mneethiraj commented on PR #252:
URL: https://github.com/apache/ranger/pull/252#issuecomment-2224253321

   @sercanCyberVision  - good catch on use of `readEntity()` in `RangerClient` 
and `RangerAdminRESTClient`. I updated these classes to deserialize the 
response using JsonUtilV2 methods. `YarnClient` uses `readEntity()` to read the 
response as String; such usage doesn't require using Jackson Json conversion.
   
   One more issue I noticed was that the patch replaced NOT_NULL to NOT_EMPTY 
in several classes; this caused errors in UI as the response from server didn't 
include empty collection values - [] or {}.
   
   I updated the patch to address above issues and will attach in RANGER-4225.


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