Hive
  1. Hive
  2. HIVE-6249

HiveServer2 JDBC SSL binary client transport should not call a TTransport#open

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.13.0
    • Fix Version/s: 0.13.0
    • Component/s: HiveServer2, JDBC
    • Labels:
      None

      Description

      Thrift's TSSLTransportFactory#getClientSocket already returns an open socket. HiveConnection#openTransport should call open only on sockets which are not bound.

      1. HIVE-6249.1.patch
        1 kB
        Vaibhav Gumashta

        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/12624210/HIVE-6249.1.patch

        SUCCESS: +1 4943 tests passed

        Test results: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/979/testReport
        Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/979/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: 12624210

        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/12624210/HIVE-6249.1.patch SUCCESS: +1 4943 tests passed Test results: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/979/testReport Console output: http://bigtop01.cloudera.org:8080/job/PreCommit-HIVE-Build/979/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: 12624210
        Hide
        Vaibhav Gumashta added a comment -
        Show
        Vaibhav Gumashta added a comment - cc Thejas M Nair
        Hide
        Vaibhav Gumashta added a comment -
        Show
        Vaibhav Gumashta added a comment - Review board: https://reviews.apache.org/r/17420/
        Hide
        Thejas M Nair added a comment -

        +1

        Show
        Thejas M Nair added a comment - +1
        Hide
        Thejas M Nair added a comment -

        Patch committed to trunk. Thanks for the contribution Vaibhav!

        Show
        Thejas M Nair added a comment - Patch committed to trunk. Thanks for the contribution Vaibhav!

          People

          • Assignee:
            Vaibhav Gumashta
            Reporter:
            Vaibhav Gumashta
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development