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

Specify more options when adding new kudu columns

    Details

    • Type: New Feature
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Impala 2.8.0
    • Fix Version/s: Impala 2.9.0
    • Component/s: Frontend
    • Labels:

      Description

      Currently, due to limitations in the Kudu API, Impala imposes a number of constraints on new columns:
      1. nullable columns can't have default values (KUDU-1747)
      2. no encoding/compression/block size can be specified (KUDU-1746)
      Now that Kudu jiras have been resolved, we should remove these constraints on the Impala side.

        Activity

        Hide
        mjacobs Matthew Jacobs added a comment -

        commit 3a2a380cf73c55e55f4025c008c50accda056f85
        Author: Matthew Jacobs <mj@cloudera.com>
        Date: Wed Mar 1 15:58:49 2017 -0800

        IMPALA-4616: Add missing Kudu column options

        Adds support for missing Kudu column options in ALTER TABLE
        (it was there in CREATE TABLE already):

        • encoding
        • compression
        • block_size

        Also adds support for adding nullable columns with default
        values.

        All of the above was not originally implemented due to
        limitations in the Kudu client, but have since been fixed:
        KUDU-1746, KUDU-1747

        Testing: Updates and adds relevant test cases.

        Change-Id: I96a0fce7f6bc0c086b259d3119daa72c94b0af7b
        Reviewed-on: http://gerrit.cloudera.org:8080/6220
        Reviewed-by: Marcel Kornacker <marcel@cloudera.com>
        Tested-by: Impala Public Jenkins

        Show
        mjacobs Matthew Jacobs added a comment - commit 3a2a380cf73c55e55f4025c008c50accda056f85 Author: Matthew Jacobs <mj@cloudera.com> Date: Wed Mar 1 15:58:49 2017 -0800 IMPALA-4616 : Add missing Kudu column options Adds support for missing Kudu column options in ALTER TABLE (it was there in CREATE TABLE already): encoding compression block_size Also adds support for adding nullable columns with default values. All of the above was not originally implemented due to limitations in the Kudu client, but have since been fixed: KUDU-1746 , KUDU-1747 Testing: Updates and adds relevant test cases. Change-Id: I96a0fce7f6bc0c086b259d3119daa72c94b0af7b Reviewed-on: http://gerrit.cloudera.org:8080/6220 Reviewed-by: Marcel Kornacker <marcel@cloudera.com> Tested-by: Impala Public Jenkins
        Hide
        mjacobs Matthew Jacobs added a comment -

        Dimitris Tsirogiannis this is specifically for ALTER TABLE ADD COLUMN, right? i.e. this does not apply to CREATE TABLE.

        Show
        mjacobs Matthew Jacobs added a comment - Dimitris Tsirogiannis this is specifically for ALTER TABLE ADD COLUMN, right? i.e. this does not apply to CREATE TABLE.

          People

          • Assignee:
            mjacobs Matthew Jacobs
            Reporter:
            dtsirogiannis Dimitris Tsirogiannis
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development