Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-7239

Fix bug in HiveIndexedInputFormat implementation that causes incorrect query result when input backed by Sequence/RC files

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.0
    • Fix Version/s: 2.2.0
    • Component/s: Indexing
    • Labels:
      None

      Description

      In case of sequence files, it's crucial that splits are calculated around the boundaries enforced by the input sequence file. However by default hadoop creates input splits depending on the configuration parameters which may not match the boundaries for the input sequence file. Hive provides HiveIndexedInputFormat that provides extra logic and recalculates the split boundaries for each split depending on the sequence file's boundaries.

      However we noticed this behavior of "over" reporting from data backed by sequence file. We've a sample data on which we experimented and fixed this bug, we have verified this fix by comparing the query output for input being sequence file format, rc file and regular format. However we have not able to find the right place to include this as a unit test that would execute as part of hive tests. We tried writing a "clientpositive" test as part of ql module but the output seems quite verbose and i couldn't interpret it that well. Can someone please review this change and guide on how to write a test that will execute as part of Hive testing?

        Attachments

        1. HIVE-7239.4.patch
          36 kB
          Illya Yalovyy
        2. HIVE-7239.3.patch
          36 kB
          Illya Yalovyy
        3. HIVE-7239.2.patch
          36 kB
          Illya Yalovyy
        4. HIVE-7239.patch
          7 kB
          Sumit Kumar

          Issue Links

            Activity

              People

              • Assignee:
                yalovyyi Illya Yalovyy
                Reporter:
                ksumit Sumit Kumar
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: