Skip to content

HADOOP: fix 'occured' -> 'occurred' typos in 3 files#8439

Merged
ajfabbri merged 1 commit into
apache:trunkfrom
SAY-5:fix-occured-occurred-typo
Apr 17, 2026
Merged

HADOOP: fix 'occured' -> 'occurred' typos in 3 files#8439
ajfabbri merged 1 commit into
apache:trunkfrom
SAY-5:fix-occured-occurred-typo

Conversation

@SAY-5
Copy link
Copy Markdown
Contributor

@SAY-5 SAY-5 commented Apr 17, 2026

Trivial spelling fix in three source files — occuredoccurred.

All occurrences are in code comments / Javadoc:

  • hadoop-hdfs-native-client/.../libhdfs/os/thread_local_storage.h — two struct field comments describing the last exception stack trace / root cause.
  • hadoop-aws/.../AbstractS3ACommitter.java@throws IOException Javadoc on maybeSaveSummary.
  • hadoop-mapreduce-client-core/.../ManifestCommitter.java@throws IOException Javadoc on maybeSaveSummary.

No functional changes.

- hadoop-hdfs-native-client/.../thread_local_storage.h: two struct
  field comments in the libhdfs thread-local-storage header.
- hadoop-aws/.../AbstractS3ACommitter.java: @throws Javadoc tag on
  maybeSaveSummary.
- hadoop-mapreduce-client-core/.../ManifestCommitter.java: @throws
  Javadoc tag on maybeSaveSummary.

No functional changes.

Signed-off-by: Sai Asish Y <say.apm35@gmail.com>
@hadoop-yetus
Copy link
Copy Markdown

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 8m 30s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗 mvndep 1m 46s Maven dependency ordering for branch
+1 💚 mvninstall 25m 19s trunk passed
+1 💚 compile 10m 10s trunk passed with JDK Red Hat, Inc.-21.0.10+7-LTS
+1 💚 compile 10m 18s trunk passed with JDK Red Hat, Inc.-17.0.18+8-LTS
+1 💚 checkstyle 3m 51s trunk passed
+1 💚 mvnsite 4m 24s trunk passed
+1 💚 javadoc 4m 12s trunk passed with JDK Red Hat, Inc.-21.0.10+7-LTS
+1 💚 javadoc 4m 9s trunk passed with JDK Red Hat, Inc.-17.0.18+8-LTS
+0 🆗 spotbugs 1m 22s branch/hadoop-hdfs-project/hadoop-hdfs-native-client no spotbugs output file (spotbugsXml.xml)
+1 💚 shadedclient 18m 31s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 40s Maven dependency ordering for patch
+1 💚 mvninstall 3m 27s the patch passed
+1 💚 compile 8m 55s the patch passed with JDK Red Hat, Inc.-21.0.10+7-LTS
+1 💚 cc 8m 55s the patch passed
+1 💚 golang 8m 55s the patch passed
+1 💚 javac 8m 55s the patch passed
+1 💚 compile 10m 10s the patch passed with JDK Red Hat, Inc.-17.0.18+8-LTS
+1 💚 cc 10m 10s the patch passed
+1 💚 golang 10m 10s the patch passed
+1 💚 javac 10m 10s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 3m 54s the patch passed
+1 💚 mvnsite 4m 26s the patch passed
+1 💚 javadoc 4m 9s the patch passed with JDK Red Hat, Inc.-21.0.10+7-LTS
+1 💚 javadoc 4m 8s the patch passed with JDK Red Hat, Inc.-17.0.18+8-LTS
+0 🆗 spotbugs 1m 19s hadoop-hdfs-project/hadoop-hdfs-native-client has no data from spotbugs
+1 💚 shadedclient 18m 35s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 7m 24s hadoop-hdfs-native-client in the patch passed.
+1 💚 unit 7m 11s hadoop-mapreduce-client-core in the patch passed.
+1 💚 unit 3m 31s hadoop-aws in the patch passed.
+1 💚 asflicense 1m 41s The patch does not generate ASF License warnings.
186m 40s
Subsystem Report/Notes
Docker ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8439/1/artifact/out/Dockerfile
GITHUB PR #8439
Optional Tests dupname asflicense compile cc mvnsite javac unit codespell detsecrets golang javadoc mvninstall shadedclient spotbugs checkstyle
uname Linux aef3b31dd5c8 5.15.0-173-generic #183-Ubuntu SMP Fri Mar 6 13:29:34 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 51f31c6
Default Java Red Hat, Inc.-17.0.18+8-LTS
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-21.0.10.0.7-1.el8.x86_64:Red Hat, Inc.-21.0.10+7-LTS /usr/lib/jvm/java-17-openjdk-17.0.18.0.8-1.el8.x86_64:Red Hat, Inc.-17.0.18+8-LTS
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8439/1/testReport/
Max. process+thread count 1575 (vs. ulimit of 10000)
modules C: hadoop-hdfs-project/hadoop-hdfs-native-client hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core hadoop-tools/hadoop-aws U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8439/1/console
versions git=2.43.7 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link
Copy Markdown

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 6m 46s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗 mvndep 0m 19s Maven dependency ordering for branch
+1 💚 mvninstall 19m 22s trunk passed
+1 💚 compile 8m 46s trunk passed
+1 💚 checkstyle 2m 44s trunk passed
+1 💚 mvnsite 2m 5s trunk passed
+1 💚 javadoc 1m 48s trunk passed
+0 🆗 spotbugs 0m 32s branch/hadoop-hdfs-project/hadoop-hdfs-native-client no spotbugs output file (spotbugsXml.xml)
+1 💚 shadedclient 16m 13s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 18s Maven dependency ordering for patch
+1 💚 mvninstall 2m 37s the patch passed
+1 💚 compile 8m 16s the patch passed
+1 💚 cc 8m 16s the patch passed
+1 💚 golang 8m 17s the patch passed
+1 💚 javac 8m 16s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 2m 47s the patch passed
+1 💚 mvnsite 2m 4s the patch passed
+1 💚 javadoc 1m 49s the patch passed
+0 🆗 spotbugs 0m 32s hadoop-hdfs-project/hadoop-hdfs-native-client has no data from spotbugs
+1 💚 shadedclient 16m 13s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 6m 15s hadoop-hdfs-native-client in the patch passed.
+1 💚 unit 6m 23s hadoop-mapreduce-client-core in the patch passed.
+1 💚 unit 2m 43s hadoop-aws in the patch passed.
+1 💚 asflicense 0m 48s The patch does not generate ASF License warnings.
119m 4s
Subsystem Report/Notes
Docker ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8439/1/artifact/out/Dockerfile
GITHUB PR #8439
Optional Tests dupname asflicense compile cc mvnsite javac unit codespell detsecrets golang javadoc mvninstall shadedclient spotbugs checkstyle
uname Linux 116724ea8aff 5.15.0-173-generic #183-Ubuntu SMP Fri Mar 6 13:29:34 UTC 2026 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 51f31c6
Default Java Debian-25.0.2+10-Debian-1deb13u2
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8439/1/testReport/
Max. process+thread count 1581 (vs. ulimit of 10000)
modules C: hadoop-hdfs-project/hadoop-hdfs-native-client hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core hadoop-tools/hadoop-aws U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8439/1/console
versions git=2.47.3 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@hadoop-yetus
Copy link
Copy Markdown

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 22s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 1s codespell was not available.
+0 🆗 detsecrets 0m 1s detect-secrets was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗 mvndep 0m 18s Maven dependency ordering for branch
+1 💚 mvninstall 19m 31s trunk passed
+1 💚 compile 8m 42s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 compile 8m 59s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 checkstyle 2m 56s trunk passed
+1 💚 mvnsite 2m 7s trunk passed
+1 💚 javadoc 1m 47s trunk passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 1m 47s trunk passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+0 🆗 spotbugs 0m 33s branch/hadoop-hdfs-project/hadoop-hdfs-native-client no spotbugs output file (spotbugsXml.xml)
+1 💚 shadedclient 16m 24s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 17s Maven dependency ordering for patch
+1 💚 mvninstall 2m 36s the patch passed
+1 💚 compile 8m 19s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 cc 8m 19s the patch passed
+1 💚 golang 8m 19s the patch passed
+1 💚 javac 8m 19s the patch passed
+1 💚 compile 9m 0s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+1 💚 cc 9m 0s the patch passed
+1 💚 golang 9m 0s the patch passed
+1 💚 javac 9m 0s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 checkstyle 2m 55s the patch passed
+1 💚 mvnsite 1m 33s the patch passed
+1 💚 javadoc 1m 18s the patch passed with JDK Ubuntu-21.0.10+7-Ubuntu-124.04
+1 💚 javadoc 1m 28s the patch passed with JDK Ubuntu-17.0.18+8-Ubuntu-124.04.1
+0 🆗 spotbugs 0m 23s hadoop-hdfs-project/hadoop-hdfs-native-client has no data from spotbugs
+1 💚 shadedclient 18m 35s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 unit 6m 15s hadoop-hdfs-native-client in the patch passed.
+1 💚 unit 6m 23s hadoop-mapreduce-client-core in the patch passed.
+1 💚 unit 2m 49s hadoop-aws in the patch passed.
+1 💚 asflicense 0m 50s The patch does not generate ASF License warnings.
136m 49s
Subsystem Report/Notes
Docker ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8439/1/artifact/out/Dockerfile
GITHUB PR #8439
Optional Tests dupname asflicense compile cc mvnsite javac unit codespell detsecrets golang javadoc mvninstall shadedclient spotbugs checkstyle
uname Linux 1755f3842842 5.15.0-173-generic #183-Ubuntu SMP Fri Mar 6 13:29:34 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 51f31c6
Default Java Ubuntu-17.0.18+8-Ubuntu-124.04.1
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.10+7-Ubuntu-124.04 /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.18+8-Ubuntu-124.04.1
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8439/1/testReport/
Max. process+thread count 1680 (vs. ulimit of 10000)
modules C: hadoop-hdfs-project/hadoop-hdfs-native-client hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core hadoop-tools/hadoop-aws U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8439/1/console
versions git=2.43.0 maven=3.9.11 spotbugs=4.9.7
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

Copy link
Copy Markdown
Contributor

@ajfabbri ajfabbri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Easy review. +1

@ajfabbri
Copy link
Copy Markdown
Contributor

Thanks for your contribution. Only CI failure was test4tests which does not apply here. Merging to trunk

@ajfabbri ajfabbri merged commit c1fdf58 into apache:trunk Apr 17, 2026
1 of 3 checks passed
@pan3793
Copy link
Copy Markdown
Member

pan3793 commented Apr 19, 2026

@ajfabbri, do we want to accept minor changes such as this PR without a JIRA ticket, like MINOR. This is the title of pull request?

@ajfabbri
Copy link
Copy Markdown
Contributor

Great question. I wondered the same thing. Do we have an official prefix that we like to use for trivial commits like these? It's not worth filing an issue over but would like to be consistent.

@pan3793
Copy link
Copy Markdown
Member

pan3793 commented Apr 19, 2026

I was asked to create a JIRA ticket for even a minor change in hadoop project, but other projects usually allow MINOR alternative, maybe we need a vote to promote "MINOR" as an official prefix?

@ajfabbri
Copy link
Copy Markdown
Contributor

I was asked to create a JIRA ticket for even a minor change in hadoop project, but other projects usually allow MINOR alternative, maybe we need a vote to promote "MINOR" as an official prefix?

You are correct. I started a [discuss] thread on common-dev. Thanks for the reminder!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants