[GitHub] [hadoop] szilard-nemeth commented on pull request #5332: YARN-11041. Replace all occurences of queuePath with the new QueuePath class - followup

["szilard-nemeth (via GitHub)" <gi...@apache.org>]

szilard-nemeth commented on PR #5332:
URL: https://github.com/apache/hadoop/pull/5332#issuecomment-1530671809

   Thanks @p-szucs for this patch, good job.
   Added some review comments.
   Please don't rebase and squash existing commits, just point out new commits to review with your fixes. No way I will review this monster once again :D 


-- 
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: common-issues-unsubscribe@hadoop.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org