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

   ### Description of PR
   filter() method inside the ClientJerseyRetryFilter class, calls 
checkUrlConnectivity() in a while loop, If there is any issue with 
connectivity, it decrements the leftRetries till it becomes 0 and throws an 
Exception, but if it succeeds it keeps on spinning inside the infinite while 
loop, so added a return after checkUrlConnectivity().
   
   After fixing the above issue, the yarn logs command was throwing below error.
   
   Can not find any log file matching the pattern: [ALL] for the container: 
container_id within the application: app_id
   
   The XML response of /ws/v1/node/containers/<container_id>/logs is
   `<containerLogsInfoes>
   <containerLogsInfo>
   <containerLogInfo>
   <fileName>stdout</fileName>
   <fileSize>0</fileSize>
   <lastModifiedTime>Wed Nov 05 19:41:59 +0530 2025</lastModifiedTime>
   </containerLogInfo>
   <containerLogInfo>
   <fileName>prelaunch.out</fileName>
   <fileSize>100</fileSize>
   <lastModifiedTime>Wed Nov 05 19:41:59 +0530 2025</lastModifiedTime>
   </containerLogInfo>
   <containerLogInfo>
   <fileName>launch_container.sh</fileName>
   <fileSize>6125</fileSize>
   <lastModifiedTime>Wed Nov 05 19:41:59 +0530 2025</lastModifiedTime>
   </containerLogInfo>
   <containerLogInfo>
   <fileName>directory.info</fileName>
   <fileSize>2336</fileSize>
   <lastModifiedTime>Wed Nov 05 19:41:59 +0530 2025</lastModifiedTime>
   </containerLogInfo>
   <containerLogInfo>
   <fileName>prelaunch.err</fileName>
   <fileSize>0</fileSize>
   <lastModifiedTime>Wed Nov 05 19:41:59 +0530 2025</lastModifiedTime>
   </containerLogInfo>
   <containerLogInfo>
   <fileName>syslog</fileName>
   <fileSize>77682</fileSize>
   <lastModifiedTime>Wed Nov 05 19:42:21 +0530 2025</lastModifiedTime>
   </containerLogInfo>
   <containerLogInfo>
   <fileName>stderr</fileName>
   <fileSize>0</fileSize>
   <lastModifiedTime>Wed Nov 05 19:41:59 +0530 2025</lastModifiedTime>
   </containerLogInfo>
   <logAggregationType>LOCAL</logAggregationType>
   <containerId>container_id</containerId>
   <nodeId>hostname:42923</nodeId>
   </containerLogsInfo>
   </containerLogsInfoes>`
   
   So in getContainerLogFiles(), an additional 
getJSONObject("containerLogsInfoes") has been added to ensure the XML response 
is parsed properly
   
   
   ### How was this patch tested?
   Validated yarn logs -applicationId <appID> for running and completed 
applications.
   
   ### For code changes:
   
   - [ ] Does the title or this PR starts with the corresponding JIRA issue id 
(e.g. 'HADOOP-17799. Your PR title ...')?
   - [ ] Object storage: have the integration tests been executed and the 
endpoint declared according to the connector-specific documentation?
   - [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?
   - [ ] If applicable, have you updated the `LICENSE`, `LICENSE-binary`, 
`NOTICE-binary` files?
   
   


-- 
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