Uploaded image for project: 'ORC'
  1. ORC
  2. ORC-1485 Enable checkstyle checks for test classes
  3. ORC-1486

Fix checkstyle violations for tests in orc-core module

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.0.0
    • 2.0.0
    • Java
    • None

    Description

      Currently orc-core module classes have checkstyle violation (1241). We need to fix them to allow enablement of checkstyle for test code

       

      [INFO] ------------------------------------------------------------------------
      [INFO] Reactor Summary for Apache ORC 2.0.0-SNAPSHOT:
      [INFO] 
      [INFO] Apache ORC ......................................... SUCCESS [  3.010 s]
      [INFO] ORC Shims .......................................... SUCCESS [  4.234 s]
      [INFO] ORC Core ........................................... FAILURE [01:55 min]
      [INFO] ORC MapReduce ...................................... SKIPPED
      [INFO] ORC Tools .......................................... SKIPPED
      [INFO] ORC Examples ....................................... SKIPPED
      [INFO] ------------------------------------------------------------------------
      [INFO] BUILD FAILURE
      [INFO] ------------------------------------------------------------------------
      [INFO] Total time:  02:02 min
      [INFO] Finished at: 2023-08-19T10:56:26-07:00
      [INFO] ------------------------------------------------------------------------
      [ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.3.0:check (default) on project orc-core: You have 1241 Checkstyle violations. -> [Help 1] 

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            mysticlama Ashish
            mysticlama Ashish
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment