|
| 1 | +diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/serializer/avro/TestAvroSerialization.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/serializer/avro/TestAvroSerialization.java |
| 2 | +index b2d2a8d100..97e281ba85 100644 |
| 3 | +--- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/serializer/avro/TestAvroSerialization.java |
| 4 | ++++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/serializer/avro/TestAvroSerialization.java |
| 5 | +@@ -33,7 +33,7 @@ public class TestAvroSerialization { |
| 6 | + @Test |
| 7 | + public void testSpecific() throws Exception { |
| 8 | + AvroRecord before = new AvroRecord(); |
| 9 | +- before.intField = 5; |
| 10 | ++ before.setIntField(5); |
| 11 | + AvroRecord after = SerializationTestUtil.testSerialization(conf, before); |
| 12 | + assertEquals(before, after); |
| 13 | + } |
| 14 | +diff --git a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobQueueChangeEvent.java b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobQueueChangeEvent.java |
| 15 | +index 66f3781239..aa70bbff62 100644 |
| 16 | +--- a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobQueueChangeEvent.java |
| 17 | ++++ b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobQueueChangeEvent.java |
| 18 | +@@ -31,8 +31,8 @@ public class JobQueueChangeEvent implements HistoryEvent { |
| 19 | + private JobQueueChange datum = new JobQueueChange(); |
| 20 | + |
| 21 | + public JobQueueChangeEvent(JobID id, String queueName) { |
| 22 | +- datum.jobid = new Utf8(id.toString()); |
| 23 | +- datum.jobQueueName = new Utf8(queueName); |
| 24 | ++ datum.setJobid(new Utf8(id.toString())); |
| 25 | ++ datum.setJobQueueName(new Utf8(queueName)); |
| 26 | + } |
| 27 | + |
| 28 | + JobQueueChangeEvent() { } |
| 29 | +@@ -54,13 +54,13 @@ public class JobQueueChangeEvent implements HistoryEvent { |
| 30 | + |
| 31 | + /** Get the Job ID */ |
| 32 | + public JobID getJobId() { |
| 33 | +- return JobID.forName(datum.jobid.toString()); |
| 34 | ++ return JobID.forName(datum.getJobid().toString()); |
| 35 | + } |
| 36 | + |
| 37 | + /** Get the new Job queue name */ |
| 38 | + public String getJobQueueName() { |
| 39 | +- if (datum.jobQueueName != null) { |
| 40 | +- return datum.jobQueueName.toString(); |
| 41 | ++ if (datum.getJobQueueName() != null) { |
| 42 | ++ return datum.getJobQueueName().toString(); |
| 43 | + } |
| 44 | + return null; |
| 45 | + } |
| 46 | +diff --git a/hadoop-project/pom.xml b/hadoop-project/pom.xml |
| 47 | +index bcba56eced..2d52e3ebb1 100644 |
| 48 | +--- a/hadoop-project/pom.xml |
| 49 | ++++ b/hadoop-project/pom.xml |
| 50 | +@@ -37,7 +37,7 @@ |
| 51 | + <!--Whether to proceed to next module if any test failures exist--> |
| 52 | + <maven.test.failure.ignore>true</maven.test.failure.ignore> |
| 53 | + <maven.test.redirectTestOutputToFile>true</maven.test.redirectTestOutputToFile> |
| 54 | +- <jetty.version>9.4.53.v20231009</jetty.version> |
| 55 | ++ <jetty.version>9.4.56.v20240826</jetty.version> |
| 56 | + <test.exclude>_</test.exclude> |
| 57 | + <test.exclude.pattern>_</test.exclude.pattern> |
| 58 | + |
| 59 | +@@ -63,7 +63,7 @@ |
| 60 | + <java.security.egd>file:///dev/urandom</java.security.egd> |
| 61 | + |
| 62 | + <!-- avro version --> |
| 63 | +- <avro.version>1.9.2</avro.version> |
| 64 | ++ <avro.version>1.11.4</avro.version> |
| 65 | + |
| 66 | + <!-- jersey version --> |
| 67 | + <jersey.version>1.19.4</jersey.version> |
| 68 | +@@ -108,7 +108,7 @@ |
| 69 | + <findbugs.version>3.0.5</findbugs.version> |
| 70 | + <dnsjava.version>3.6.1</dnsjava.version> |
| 71 | + |
| 72 | +- <guava.version>27.0-jre</guava.version> |
| 73 | ++ <guava.version>32.0.1-jre</guava.version> |
| 74 | + <guice.version>4.2.3</guice.version> |
| 75 | + |
| 76 | + <bouncycastle.version>1.78.1</bouncycastle.version> |
| 77 | +@@ -143,7 +143,7 @@ |
| 78 | + <jna.version>5.2.0</jna.version> |
| 79 | + <gson.version>2.9.0</gson.version> |
| 80 | + <metrics.version>3.2.4</metrics.version> |
| 81 | +- <netty4.version>4.1.100.Final</netty4.version> |
| 82 | ++ <netty4.version>4.1.108.Final</netty4.version> |
| 83 | + <snappy-java.version>1.1.10.4</snappy-java.version> |
| 84 | + <lz4-java.version>1.7.1</lz4-java.version> |
| 85 | + |
| 86 | +diff --git a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JobBuilder.java b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JobBuilder.java |
| 87 | +index 1213e6a46f..603b248f6e 100644 |
| 88 | +--- a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JobBuilder.java |
| 89 | ++++ b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JobBuilder.java |
| 90 | +@@ -460,7 +460,7 @@ public class JobBuilder { |
| 91 | + } |
| 92 | + task.setFinishTime(event.getFinishTime()); |
| 93 | + task.setTaskStatus(getPre21Value(event.getTaskStatus())); |
| 94 | +- task.incorporateCounters(((TaskFinished) event.getDatum()).counters); |
| 95 | ++ task.incorporateCounters(((TaskFinished) event.getDatum()).getCounters()); |
| 96 | + } |
| 97 | + |
| 98 | + private void processTaskFailedEvent(TaskFailedEvent event) { |
| 99 | +@@ -472,7 +472,7 @@ public class JobBuilder { |
| 100 | + task.setFinishTime(event.getFinishTime()); |
| 101 | + task.setTaskStatus(getPre21Value(event.getTaskStatus())); |
| 102 | + TaskFailed t = (TaskFailed)(event.getDatum()); |
| 103 | +- task.putDiagnosticInfo(t.error.toString()); |
| 104 | ++ task.putDiagnosticInfo(t.getError().toString()); |
| 105 | + // killed task wouldn't have failed attempt. |
| 106 | + if (t.getFailedDueToAttempt() != null) { |
| 107 | + task.putFailedDueToAttemptId(t.getFailedDueToAttempt().toString()); |
| 108 | +@@ -542,7 +542,7 @@ public class JobBuilder { |
| 109 | + } |
| 110 | + attempt.setFinishTime(event.getFinishTime()); |
| 111 | + attempt |
| 112 | +- .incorporateCounters(((TaskAttemptFinished) event.getDatum()).counters); |
| 113 | ++ .incorporateCounters(((TaskAttemptFinished) event.getDatum()).getCounters()); |
| 114 | + } |
| 115 | + |
| 116 | + private void processReduceAttemptFinishedEvent( |
| 117 | +@@ -568,7 +568,7 @@ public class JobBuilder { |
| 118 | + attempt.setShuffleFinished(event.getShuffleFinishTime()); |
| 119 | + attempt.setSortFinished(event.getSortFinishTime()); |
| 120 | + attempt |
| 121 | +- .incorporateCounters(((ReduceAttemptFinished) event.getDatum()).counters); |
| 122 | ++ .incorporateCounters(((ReduceAttemptFinished) event.getDatum()).getCounters()); |
| 123 | + attempt.arraySetClockSplits(event.getClockSplits()); |
| 124 | + attempt.arraySetCpuUsages(event.getCpuUsages()); |
| 125 | + attempt.arraySetVMemKbytes(event.getVMemKbytes()); |
| 126 | +@@ -596,7 +596,7 @@ public class JobBuilder { |
| 127 | + // is redundant, but making this will add future-proofing. |
| 128 | + attempt.setFinishTime(event.getFinishTime()); |
| 129 | + attempt |
| 130 | +- .incorporateCounters(((MapAttemptFinished) event.getDatum()).counters); |
| 131 | ++ .incorporateCounters(((MapAttemptFinished) event.getDatum()).getCounters()); |
| 132 | + attempt.arraySetClockSplits(event.getClockSplits()); |
| 133 | + attempt.arraySetCpuUsages(event.getCpuUsages()); |
| 134 | + attempt.arraySetVMemKbytes(event.getVMemKbytes()); |
| 135 | +@@ -661,11 +661,11 @@ public class JobBuilder { |
| 136 | + |
| 137 | + JobFinished job = (JobFinished)event.getDatum(); |
| 138 | + Map<String, Long> countersMap = |
| 139 | +- JobHistoryUtils.extractCounters(job.totalCounters); |
| 140 | ++ JobHistoryUtils.extractCounters(job.getTotalCounters()); |
| 141 | + result.putTotalCounters(countersMap); |
| 142 | +- countersMap = JobHistoryUtils.extractCounters(job.mapCounters); |
| 143 | ++ countersMap = JobHistoryUtils.extractCounters(job.getMapCounters()); |
| 144 | + result.putMapCounters(countersMap); |
| 145 | +- countersMap = JobHistoryUtils.extractCounters(job.reduceCounters); |
| 146 | ++ countersMap = JobHistoryUtils.extractCounters(job.getReduceCounters()); |
| 147 | + result.putReduceCounters(countersMap); |
| 148 | + } |
| 149 | + |
| 150 | +diff --git a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JobHistoryUtils.java b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JobHistoryUtils.java |
| 151 | +index 6ae87bbd40..34ef95f337 100644 |
| 152 | +--- a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JobHistoryUtils.java |
| 153 | ++++ b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/JobHistoryUtils.java |
| 154 | +@@ -157,9 +157,9 @@ public class JobHistoryUtils { |
| 155 | + static Map<String, Long> extractCounters(JhCounters counters) { |
| 156 | + Map<String, Long> countersMap = new HashMap<String, Long>(); |
| 157 | + if (counters != null) { |
| 158 | +- for (JhCounterGroup group : counters.groups) { |
| 159 | +- for (JhCounter counter : group.counts) { |
| 160 | +- countersMap.put(counter.name.toString(), counter.value); |
| 161 | ++ for (JhCounterGroup group : counters.getGroups()) { |
| 162 | ++ for (JhCounter counter : group.getCounts()) { |
| 163 | ++ countersMap.put(counter.getName().toString(), counter.getValue()); |
| 164 | + } |
| 165 | + } |
| 166 | + } |
| 167 | +diff --git a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/LoggedTask.java b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/LoggedTask.java |
| 168 | +index 4ae33a7661..2308e58690 100644 |
| 169 | +--- a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/LoggedTask.java |
| 170 | ++++ b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/LoggedTask.java |
| 171 | +@@ -268,11 +268,11 @@ public class LoggedTask implements DeepCompare { |
| 172 | + String counterName) { |
| 173 | + counterName = canonicalizeCounterName(counterName); |
| 174 | + |
| 175 | +- for (JhCounterGroup group : counters.groups) { |
| 176 | +- for (JhCounter counter : group.counts) { |
| 177 | ++ for (JhCounterGroup group : counters.getGroups()) { |
| 178 | ++ for (JhCounter counter : group.getCounts()) { |
| 179 | + if (counterName |
| 180 | +- .equals(canonicalizeCounterName(counter.name.toString()))) { |
| 181 | +- thunk.set(counter.value); |
| 182 | ++ .equals(canonicalizeCounterName(counter.getName().toString()))) { |
| 183 | ++ thunk.set(counter.getValue()); |
| 184 | + return; |
| 185 | + } |
| 186 | + } |
| 187 | +diff --git a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/LoggedTaskAttempt.java b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/LoggedTaskAttempt.java |
| 188 | +index 5c6abd372c..fae53b2926 100644 |
| 189 | +--- a/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/LoggedTaskAttempt.java |
| 190 | ++++ b/hadoop-tools/hadoop-rumen/src/main/java/org/apache/hadoop/tools/rumen/LoggedTaskAttempt.java |
| 191 | +@@ -636,11 +636,11 @@ public class LoggedTaskAttempt implements DeepCompare { |
| 192 | + String counterName) { |
| 193 | + counterName = canonicalizeCounterName(counterName); |
| 194 | + |
| 195 | +- for (JhCounterGroup group : counters.groups) { |
| 196 | +- for (JhCounter counter : group.counts) { |
| 197 | ++ for (JhCounterGroup group : counters.getGroups()) { |
| 198 | ++ for (JhCounter counter : group.getCounts()) { |
| 199 | + if (counterName |
| 200 | +- .equals(canonicalizeCounterName(counter.name.toString()))) { |
| 201 | +- thunk.set(counter.value); |
| 202 | ++ .equals(canonicalizeCounterName(counter.getName().toString()))) { |
| 203 | ++ thunk.set(counter.getValue()); |
| 204 | + return; |
| 205 | + } |
| 206 | + } |
| 207 | +@@ -769,4 +769,4 @@ public class LoggedTaskAttempt implements DeepCompare { |
| 208 | + compare1(vMemKbytes, other.vMemKbytes, loc, "vMemKbytes"); |
| 209 | + compare1(physMemKbytes, other.physMemKbytes, loc, "physMemKbytes"); |
| 210 | + } |
| 211 | +-} |
| 212 | +\ No newline at end of file |
| 213 | ++} |
0 commit comments