muskan1012 commented on PR #6968:
URL: https://github.com/apache/hadoop/pull/6968#issuecomment-2409992934

   > code is good, but the import ordering is all mixed up because of how the 
mockito import was moved to the new one.
   > 
   > please review and fix
   > 
   > * position of org.apache import in static and class imports
   > * blank likes between blocks
   > 
   > fix this and we are good. (oh, and you'll remember next time: I worry a 
lot about imports because they make backporting a nightmare. Save us both time 
and predict what I'll complain about before submitting patches)
   
   Sure @steveloughran, Noted it. Thank you for reviewing it!!
   


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