[ 
https://issues.apache.org/jira/browse/MAPREDUCE-7550?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18130818#comment-18130818
 ] 

ASF GitHub Bot commented on MAPREDUCE-7550:
-------------------------------------------

luoyuan3471 opened a new pull request, #8859:
URL: https://github.com/apache/hadoop/pull/8859

   ### Description of PR
   
   Fixes [MAPREDUCE-7550](https://issues.apache.org/jira/browse/MAPREDUCE-7550).
   
   This is one documentation-only patch in hadoop-mapreduce-client-core, with 
13 text corrections across 9 files:
   
   - Framework deployment documentation: remove the duplicate "is" and correct 
"accomodate" to "accommodate" in DistributedCacheDeploy.md.vm and 
MapredCommands.md.
   - Configuration descriptions: correct two instances of "overriden" to 
"overridden" in mapred-default.xml descriptions.
   - Aggregation and reducer Javadocs: correct "reduce/combiner phrase" to 
"reduce/combiner phase" and add missing sentence spaces in the mapred and 
mapreduce APIs.
   
   Configuration names and values, commands, API names, executable code, and 
existing end-of-file newline conventions are unchanged. No runtime behavior 
changes are included.
   
   ### How was this patch tested?
   
   - `git diff --check` passed.
   - `xmllint --noout` on mapred-default.xml passed.
   - Parsed and compared all non-description property fields in 
mapred-default.xml against the base; they are identical.
   - Compared every changed file against the base using the exact reviewed text 
substitutions and verified that no unrelated bytes changed. Existing 
final-newline conventions were preserved.
   - Manually reviewed all Java hunks to confirm they only modify Javadoc text.
   - Maven builds and unit tests were not run locally. No new functional tests 
are needed because this patch only corrects documentation text. Existing build, 
Javadoc, and unit-test checks remain applicable.
   
   If `test4tests` reports the absence of modified test files, please review 
that item as not applicable to this documentation-only patch. This is not a 
request to waive actual build or test failures.
   
   ### For code changes:
   
   - [x] The PR title starts with the corresponding JIRA issue id.
   - [ ] Object storage integration tests (not applicable).
   - [ ] Dependency licensing (not applicable; no new dependencies).
   - [ ] LICENSE, LICENSE-binary, or NOTICE-binary updates (not applicable).
   
   ### AI Tooling
   
   - [x] Contains content generated by Codex.
   - [x] My use of AI contributions follows the ASF legal policy: 
https://www.apache.org/legal/generative-tooling.html




> Fix typos in MapReduce documentation and Javadocs
> -------------------------------------------------
>
>                 Key: MAPREDUCE-7550
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-7550
>             Project: Hadoop Map/Reduce
>          Issue Type: Task
>          Components: documentation
>            Reporter: Yuan Luo
>            Priority: Minor
>
> Several spelling and sentence-spacing errors remain in 
> hadoop-mapreduce-client-core on current trunk.
> - Framework deployment documentation: remove the duplicate "is" in 
> DistributedCacheDeploy.md.vm and correct "accomodate" to "accommodate" there 
> and in MapredCommands.md.
> - Configuration descriptions: correct two instances of "overriden" to 
> "overridden" in mapred-default.xml descriptions.
> - Aggregation and reducer Javadocs: correct "reduce/combiner phrase" to 
> "reduce/combiner phase" and add missing spaces after sentences in the mapred 
> and mapreduce APIs.
> The patch contains 13 text corrections in 9 files, all in the MapReduce 
> client core module. It does not change configuration names or values, 
> commands, API names, executable code, or existing end-of-file newline 
> conventions.
> Validation: git diff --check passed; mapred-default.xml parses successfully; 
> all non-description property fields are unchanged; each modified file was 
> compared with the base to verify only the reviewed text substitutions. No 
> Maven build or unit tests were run locally. No new functional tests are 
> needed for this documentation-only change.



--
This message was sent by Atlassian Jira
(v10.3.26#10030026)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to