Aggarwal-Raghav commented on PR #461:
URL: https://github.com/apache/tez/pull/461#issuecomment-4038749074

   > > @abstractdog , this is ready. checkstyle is running now.
   > 
   > 
https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-461/7/artifact/out/results-checkstyle-tez-dag.txt
 "Line is longer " kind of alerts are new, can you please fix them? the rest 
looks okay, e.g. `ParameterNumber` was just improved here, because you replaced 
3 params with a context
   
   Will use intelliJ-google-java-format. Initially, I didn't do it because it 
is causing diff issue in github because of formatting, making difficult to 
review.


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

Reply via email to