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

Fix HIVE_VECTORIZATION_GROUPBY_COMPLEX_TYPES_ENABLED definition

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 4.0.0-alpha-1
    • Vectorization
    • None

    Description

      The HIVE_VECTORIZATION_GROUPBY_COMPLEX_TYPES_ENABLED is definied like this:

      HIVE_VECTORIZATION_GROUPBY_COMPLEX_TYPES_ENABLED("hive.vectorized.groupby.complex.types.enabled", true,
          "This flag should be set to true to enable group by vectorization\n" +
          "of aggregations that use complex types.\n",
          "For example, AVG uses a complex type (STRUCT) for partial aggregation results" +
          "The default value is true."),
      

      Notice that the end of the "of aggregations that use complex types.\n" line is a "," instead of "+".

      So this will end up using the following constructor using the second sentence as an "altName"

          ConfVars(String varname, Object defaultVal, String description, String altName) {
            this(varname, defaultVal, null, description, true, false, altName);
          }
      

      We should fix this.

      Attachments

        1. HIVE-23015.patch
          0.9 kB
          Peter Varga

        Activity

          People

            pvargacl Peter Varga
            pvary Peter Vary
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: