HBase
  1. HBase
  2. HBASE-10192

Empty rowkey is not allowed any more

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Not A Problem
    • Affects Version/s: 0.96.0
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      In HBase-8101, we added a check and disallowed empty rowkey. However, it is allowed in 0.94. I was wondering if there is any special reason for that, i.e., is it by intention or by mistake?

        Issue Links

          Activity

          Hide
          Enis Soztutar added a comment -

          One problem with empty row keys was that, you cannot get with en empty row key, since get will be converted into a scan, and that scan would have startKey = empty, stopKey = empty, which is a whole table scan! I remember Devaraj Das looking into that some time ago, but I was not able to find the issue.

          Show
          Enis Soztutar added a comment - One problem with empty row keys was that, you cannot get with en empty row key, since get will be converted into a scan, and that scan would have startKey = empty, stopKey = empty, which is a whole table scan! I remember Devaraj Das looking into that some time ago, but I was not able to find the issue.
          Hide
          Jimmy Xiang added a comment -

          For get, should we limit to return just one row?

          Show
          Jimmy Xiang added a comment - For get, should we limit to return just one row?
          Hide
          Devaraj Das added a comment -

          Yeah, I had made the empty row key work in HBASE-3170. In HBASE-8101 it seems the decision was to not support empty row key at all...

          Show
          Devaraj Das added a comment - Yeah, I had made the empty row key work in HBASE-3170 . In HBASE-8101 it seems the decision was to not support empty row key at all...
          Hide
          stack added a comment -

          What you want to get out of this issue Jimmy Xiang? Empty row doesn't make sense. It is a change in behavior from 0.94 but we fail fast with a decent exception thrown (unless you are using thrift as you pasted on internal irc – ugh).

          Show
          stack added a comment - What you want to get out of this issue Jimmy Xiang ? Empty row doesn't make sense. It is a change in behavior from 0.94 but we fail fast with a decent exception thrown (unless you are using thrift as you pasted on internal irc – ugh).
          Hide
          Jimmy Xiang added a comment -

          Just want to make sure we are on the same page. It is fine with me not to allow empty rowkey. Let me close it as "Not A Problem". Thanks.

          Show
          Jimmy Xiang added a comment - Just want to make sure we are on the same page. It is fine with me not to allow empty rowkey. Let me close it as "Not A Problem". Thanks.

            People

            • Assignee:
              Jimmy Xiang
              Reporter:
              Jimmy Xiang
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development