ayushtkn commented on PR #5445: URL: https://github.com/apache/hadoop/pull/5445#issuecomment-1451064227
First patch isn't something to consider itself. Second one is like very unrealistic in general, there aren't any things which we can blame above which can consume time, they are some naive if checks or so, can't take this time... I don't think we have a use case for now for this util, If my intent is just for processQueueMessages, I will expose and just shoot that directly, rather than doing the whole loop. Changing the existing uses with this would be slowing down tests, which are already above tolerable limits. Lets hold it, and observe if we get something around this in future we can circle back -- 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-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
