XMLWordPrintableJSON

    Details

      Description

      Valid table names are concatted with a '.' to a valid regions names is also a valid table name, and lead to the incorrect interpretation.

      true hfile name constraints: [0-9]+(?:_SeqID_[0-9]+)?
      region name constraints    : [a-f0-9]{16}  (but we currently just use [a-f0-9]+.)
      table name constraints     : [a-zA-Z0-9_][a-zA-Z0-9_.-]*
      

      Notice that the table name constraints completely covers all region name constraints and true hfile name constraints. (a valid hfile name is a valid part of a table name, and a valid enc region name is a valid part of a table name.

      Currently the hfilelink filename convention is <hfile><region><table>. Unfortunately, making a ref to this uses the name <hfile><region><table>.<parentregion> – the contactnation of <table>.<parentregion> is a valid table name used to get interpreted as such. The fix in HBASE-7339 requires a FileNotFoundException before going down the hfile link resolution path.

      Regardless of what we do, we need to add some char invalid for table names to the hfilelink or reference filename convention.

      Suggestion: if we changed the order of the hfile-link name we could avoid some of the confusion – <table>@<region>-<hfile>.<parentregion> (or some other separator char than '@') could be used to avoid handling on the initial filenotfoundexception but I think we'd still need a good chunk of the logic to handle opening half-storefile reader throw a hfilelink.

        Attachments

        1. HBASE-7419-v0.patch
          22 kB
          Matteo Bertozzi
        2. HBASE-7419-v1.patch
          31 kB
          Matteo Bertozzi
        3. HBASE-7419-v2.patch
          28 kB
          Matteo Bertozzi
        4. HBASE-7419-v3.patch
          28 kB
          Matteo Bertozzi

          Issue Links

            Activity

              People

              • Assignee:
                mbertozzi Matteo Bertozzi
                Reporter:
                jmhsieh Jonathan Hsieh
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: