HBase
  1. HBase
  2. HBASE-5102

Change the default value of the property "hbase.connection.per.config" to false in hbase-default.xml

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.90.6
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      The property "hbase.connection.per.config" has a default value of true in hbase-default.xml. In HConnectionManager we try to assign false as the default value if no value is specified. Better to make it uniform.
      As per Ted's suggestion making it false in the hbase-default.xml.

      1. 5102.addendum
        2 kB
        Ted Yu
      2. HBASE-5102.patch
        0.6 kB
        ramkrishna.s.vasudevan

        Activity

        ramkrishna.s.vasudevan created issue -
        ramkrishna.s.vasudevan made changes -
        Field Original Value New Value
        Attachment HBASE-5102.patch [ 12508903 ]
        ramkrishna.s.vasudevan made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12508903/HBASE-5102.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        -1 tests included. The patch doesn't appear to include any new or modified tests.
        Please justify why no new tests are needed for this patch.
        Also please list what manual steps were performed to verify this patch.

        -1 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/635//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12508903/HBASE-5102.patch against trunk revision . +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/635//console This message is automatically generated.
        Hide
        Ted Yu added a comment -

        +1 on patch.

        Show
        Ted Yu added a comment - +1 on patch.
        Ted Yu made changes -
        Hadoop Flags Reviewed [ 10343 ]
        Fix Version/s 0.90.6 [ 12319200 ]
        Hide
        Ted Yu added a comment -

        Integrated to 0.90 branch.

        Thanks for the patch Ramkrishna.

        Show
        Ted Yu added a comment - Integrated to 0.90 branch. Thanks for the patch Ramkrishna.
        Ted Yu made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Ted Yu added a comment -

        Addendum that removes stale connection in HBaseAdmin ctor

        Show
        Ted Yu added a comment - Addendum that removes stale connection in HBaseAdmin ctor
        Ted Yu made changes -
        Attachment 5102.addendum [ 12508939 ]
        Hide
        Lars Francke added a comment -

        This issue was closed as part of a bulk closing operation on 2015-11-20. All issues that have been resolved and where all fixVersions have been released have been closed (following discussions on the mailing list).

        Show
        Lars Francke added a comment - This issue was closed as part of a bulk closing operation on 2015-11-20. All issues that have been resolved and where all fixVersions have been released have been closed (following discussions on the mailing list).
        Lars Francke made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        22h 56m 1 ramkrishna.s.vasudevan 30/Dec/11 00:56
        Patch Available Patch Available Resolved Resolved
        38m 7s 1 Ted Yu 30/Dec/11 01:34
        Resolved Resolved Closed Closed
        1421d 10h 21m 1 Lars Francke 20/Nov/15 11:55

          People

          • Assignee:
            Unassigned
            Reporter:
            ramkrishna.s.vasudevan
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development