Uploaded image for project: 'IMPALA'
  1. IMPALA
  2. IMPALA-5324

Fix version check in EvalDictionaryFilters

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Impala 2.9.0
    • Fix Version/s: Impala 2.9.0
    • Component/s: Backend
    • Labels:
      None
    • Epic Color:
      ghx-label-2

      Description

      The code in EvalDictionaryFilters needs to check for version below 2.9, not 2.10.

        Activity

        Hide
        joemcdonnell Joe McDonnell added a comment -

        commit a173f70a0ccacff2de978d171d6bb42349917471
        Author: Joe McDonnell <joemcdonnell@cloudera.com>
        Date: Tue May 23 16:00:55 2017 -0700

        IMPALA-5324: Fix version check in EvalDictionaryFilters

        Due to a bootstrapping issue with the dictionary
        filtering code change, the parquet version check used in
        EvalDictionaryFilters was checking for < 2.10. However,
        the impala 2.9 parquet contains the appropriate encoding,
        so this changes the version check to be < 2.9.

        Change-Id: Icc216332171038f74ff1d2ce3066da8167095361
        Reviewed-on: http://gerrit.cloudera.org:8080/6969
        Reviewed-by: Alex Behm <alex.behm@cloudera.com>
        Tested-by: Impala Public Jenkins

        Show
        joemcdonnell Joe McDonnell added a comment - commit a173f70a0ccacff2de978d171d6bb42349917471 Author: Joe McDonnell <joemcdonnell@cloudera.com> Date: Tue May 23 16:00:55 2017 -0700 IMPALA-5324 : Fix version check in EvalDictionaryFilters Due to a bootstrapping issue with the dictionary filtering code change, the parquet version check used in EvalDictionaryFilters was checking for < 2.10. However, the impala 2.9 parquet contains the appropriate encoding, so this changes the version check to be < 2.9. Change-Id: Icc216332171038f74ff1d2ce3066da8167095361 Reviewed-on: http://gerrit.cloudera.org:8080/6969 Reviewed-by: Alex Behm <alex.behm@cloudera.com> Tested-by: Impala Public Jenkins

          People

          • Assignee:
            joemcdonnell Joe McDonnell
            Reporter:
            joemcdonnell Joe McDonnell
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development