mystic-lama opened a new pull request, #1636:
URL: https://github.com/apache/orc/pull/1636

   ### What changes were proposed in this pull request?
   The PR aims at removing checkstyle suppressions for LineLength and 
AvoidStarImport rules.
   The LineLength max property has been updated to 120 from 100 (checked the 
value in Apache Hive and Apache Arrow, both use 120)
   A suppression for src/test/org/apache/orc/tools/TestFileDump.java has been 
added. It has 2 places with long json content and it was not feasible to break 
it without affecting the readability.
   
   
   ### Why are the changes needed?
   The changes are need to keep code uniform for future commits
   
   
   ### How was this patch tested?
   Changes are limited to Unit Test code, and all test cases pass


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