vinothchandar commented on pull request #1827:
URL: https://github.com/apache/hudi/pull/1827#issuecomment-692766014


   @wangxianghu on the checkstyle change to bump up the line count to 500, I 
think we should revert to 200 as it is now. 
   I checked out a few of the issues. they can be brought within limit, by 
folding like below. 
   if not, we can turn off checkstyle selectively in that block?  
   
   
   ```
   
   public class HoodieSparkMergeHandle<T extends HoodieRecordPayload> extends
       HoodieWriteHandle<T, JavaRDD<HoodieRecord<T>>, JavaRDD<HoodieKey>, 
JavaRDD<WriteStatus>, JavaPairRDD<HoodieKey, Option<Pair<String, String>>>> {
   
   ```


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to