janosik47 opened a new pull request, #41508:
URL: https://github.com/apache/arrow/pull/41508

   ### Rationale for this change
   
   Related to https://github.com/apache/arrow/issues/35888
   
   Currently the gRPC Status.RESOURCE_EXHAUSTED exception/code is translated by 
the Java FlightServer into FlightStatusCode.INVALID_ARGUMENT and thrown to the 
client as gRPC INVALID_ARGUMENT exception.
   
   That may mislead the other party as the INVALID_ARGUMENT indicates an input 
parameters problem where in reality the backed server intention was rather 
'back off and try later'.
   
   ### What changes are included in this PR?
   
   Add the FlightStatusCode.RESOURCE_EXHAUSTED code and make sure is translated 
from/to the gRPC Status.RESOURCE_EXHAUSTED
   
   ### Are these changes tested?
   
   Unit tests included to validate the RESOURCE_EXHAUSTED translation  between 
flight and grpc codes.
   
   ### Are there any user-facing changes?
   
   No.
   
   Users may start seeing RESOURCE_EXHAUSTED instead of INVALID_ARGUMENT code. 
In both cases this is an exception seen on the client side so I am considering 
this as a _not breaking change to any public API_.
   
   Although, may have an influence in the client side flows if one decided to 
react conditionally on exception status code.
   


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

Reply via email to