FerArribas14 opened a new pull request, #397:
URL: https://github.com/apache/ranger/pull/397

   Error in ElasticSearchAuditDestination shutting down RestHighLevelClient 
client
   
   ## What changes were proposed in this pull request?
   
   RANGER-4943: Error in ElasticSearchAuditDestination shutting down 
RestHighLevelClient client
   Error encountered -> ElasticSearchAccessAuditsServiceTest -> ERROR 
org.apache.ranger.audit.provider.BaseAuditHandler – Error sending message to 
ElasticSearch java.lang.RuntimeException: Cannot execute request; I/O reactor 
status: STOPPED</resthighlevelclient>.
   
   Details:
   
   Error in using a try-with-resources block in ElasticSearchAuditDestination 
because it closes the ElasticSearch client prematurely. In a try-with-resources 
block, any resource that implements the AutoCloseable interface will 
automatically close at the end of the block, which may cause the ElasticSearch 
client to close before its use is complete.  To avoid this problem, we remove 
the try-with-resources usage for the ElasticSearch client because it already 
explicitly closes the client when it is no longer needed, specifically in 
RestHighLevelClient which implements Closeable's close method.
   
   
   ## How was this patch tested?
   
   By running ./build_ranger_using_docker.sh
   <img width="422" alt="ExampleDoc" 
src="https://github.com/user-attachments/assets/41688521-e850-4785-bfb8-ca53e5d205ca";>
   <img width="713" alt="DocsElastic" 
src="https://github.com/user-attachments/assets/9d32e071-1bcd-4884-a34a-a52eb6b00e08";>
   <img width="1738" alt="AuditsApacheRanger" 
src="https://github.com/user-attachments/assets/c7d3ccda-0a00-4f2a-be7c-00a58d41f58f";>
   
   


-- 
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: dev-unsubscr...@ranger.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to