Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.13.0
    • Component/s: None
    • Labels:
      None

      Description

      Kryo changes API's often, we should shade it so users can use their own version.

      1. HIVE-5915.patch
        0.6 kB
        Brock Noland

        Activity

        Hide
        Hive QA added a comment -

        Overall: +1 all checks pass

        Here are the results of testing the latest attachment:
        https://issues.apache.org/jira/secure/attachment/12616623/HIVE-5915.patch

        SUCCESS: +1 4449 tests passed

        Test results: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/491/testReport
        Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/491/console

        Messages:

        Executing org.apache.hive.ptest.execution.PrepPhase
        Executing org.apache.hive.ptest.execution.ExecutionPhase
        Executing org.apache.hive.ptest.execution.ReportingPhase
        

        This message is automatically generated.

        ATTACHMENT ID: 12616623

        Show
        Hive QA added a comment - Overall : +1 all checks pass Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12616623/HIVE-5915.patch SUCCESS: +1 4449 tests passed Test results: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/491/testReport Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/491/console Messages: Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase This message is automatically generated. ATTACHMENT ID: 12616623
        Hide
        Brock Noland added a comment -

        Ashutosh Chauhan would you be able to take a look at this one?

        Show
        Brock Noland added a comment - Ashutosh Chauhan would you be able to take a look at this one?
        Hide
        Ashutosh Chauhan added a comment -

        +1 On a separate note we should also consider shading our other deps like protobuf, thrift etc

        Show
        Ashutosh Chauhan added a comment - +1 On a separate note we should also consider shading our other deps like protobuf, thrift etc
        Hide
        Brock Noland added a comment -

        Agreed...

        Show
        Brock Noland added a comment - Agreed...
        Hide
        Brock Noland added a comment -

        Committed to trunk! Thank you for the review!

        Show
        Brock Noland added a comment - Committed to trunk! Thank you for the review!

          People

          • Assignee:
            Brock Noland
            Reporter:
            Brock Noland
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development