riya2305 commented on PR #8449:
URL: https://github.com/apache/hadoop/pull/8449#issuecomment-4305834606

   > That is a good idea. May I ask why it should not be in the 
ExcludeRootJsonProvider? The expected response format seems to be without the 
root element
   > 
   > Example:
   > 
   > ```
   > {
   >   "allowed": "true",
   >   "user": "hive",
   >   "diagnostics": ""
   > }
   > ```
   > 
   > Not with root rmQueueAclInfo
   > 
   > ```
   > {
   >   "rmQueueAclInfo": {
   >     "allowed": true,
   >     "user": "yarn",
   >     "diagnostics": ""
   >   }
   > }
   > ```
   > 
   > But please correct me if I am mistaken
   
   You’re not mistaken for this endpoint, which is why the fix registers it 
under CONST_UNWRAPPED_CLASSES.
   I commented about the “fallback for DTOs missing from both lists” idea which 
@K0K0V0K mentioned.


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