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

   ## Problem
   
   `AppsBlock` sends `[0, Long.MAX_VALUE]` as the application start-time range 
even
   when the web UI has no start-time restriction. `ClientRMService` consequently
   calls `RMAppImpl.getStartTime()` and acquires the application read lock for 
each
   candidate that reaches this redundant filter.
   
   ## Solution
   
   - Leave the start-time range unset when both bounds have their default 
values.
   - Preserve non-default ranges, inclusive boundaries, and invalid-range 
validation.
   - Add parameterized regression coverage and check the request after a 
protobuf
     round trip.
   
   ## Validation
   
   - JDK 17.0.8, Maven Wrapper 3.9.15: reactor compilation passed.
   - `TestAppsBlock`: 11 tests, 0 failures, 0 errors, 0 skipped.
   - With the original production code, the default and explicit full-range 
cases
     fail (11 tests, 2 failures, 0 errors, 0 skipped).
   - Fresh Surefire XML and `git diff --check` verified. Community CI is 
pending.
   - Contains content generated by Codex.
   
   ```sh
   ./mvnw -B -pl 
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common 
-am \
     -Dtest=TestAppsBlock -Dsurefire.failIfNoSpecifiedTests=false \
     -Dmaven.test.failure.ignore=false -DforkCount=1 -DskipShade test
   ```
   
   ## Related Jira
   
   [YARN-12051](https://issues.apache.org/jira/browse/YARN-12051)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


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

Reply via email to