HBase
  1. HBase
  2. HBASE-742

Column length limit is not enforced

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.1.3, 0.2.0
    • Fix Version/s: 0.2.0, 0.1.4
    • Component/s: Client
    • Labels:
      None

      Description

      HColumnDescriptor provides for a limit on column value length but it is not enforced in 0.1.3 or 0.2.0 other than in the REST and Thrift APIs. (I thought it was enforced in some earlier revision but cannot find it).

      Enforcement on the client side would be less complicated than doing it on the server side.

      1. hbase-742-v2.patch
        4 kB
        Jean-Daniel Cryans
      2. hbase-742-v1.patch
        5 kB
        Jean-Daniel Cryans

        Activity

        Hide
        Jean-Daniel Cryans added a comment -

        Correction, REST and Thrift APIs only refer to it, nothing is enforced.

        Show
        Jean-Daniel Cryans added a comment - Correction, REST and Thrift APIs only refer to it, nothing is enforced.
        Hide
        Jean-Daniel Cryans added a comment -

        First try at validating values length.

        Show
        Jean-Daniel Cryans added a comment - First try at validating values length.
        Hide
        Jean-Daniel Cryans added a comment -

        The first patch broke the build in TestTable, had to put the validation code at the HRS level. On the good side, in the first patch the getTableDescription did an RPC at each validation which was somewhat bad so having it in HRS is more direct.

        Review please.

        Show
        Jean-Daniel Cryans added a comment - The first patch broke the build in TestTable, had to put the validation code at the HRS level. On the good side, in the first patch the getTableDescription did an RPC at each validation which was somewhat bad so having it in HRS is more direct. Review please.
        Hide
        stack added a comment -

        Committed. Thanks for the patch J-D.

        Show
        stack added a comment - Committed. Thanks for the patch J-D.

          People

          • Assignee:
            Jean-Daniel Cryans
            Reporter:
            Jim Kellerman
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development