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

accept TBLPROPERTIES on CREATE TABLE/VIEW

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.5.0
    • 0.6.0
    • None
    • Reviewed

    Description

      Currently, Hive only supports ALTER TABLE t SET TBLPROPERTIES, but does not allow specification of table properties during CREATE TABLE. We should allow properties to be set at the time a table or view is created. This is useful in general, and in particular we want to use this so that storage handler properties (see HIVE-705) unrelated to serdes can be specified here rather than in SERDEPROPERTIES. See also HIVE-1144 regarding views.

      Attachments

        1. HIVE-1220.1.patch
          102 kB
          John Sichi

        Issue Links

          Activity

            jvs John Sichi added a comment -

            Once this is committed, I will update wiki docs accordingly.

            Notes for reviewers:

            • Now TBLPROPERTIES can be specified for both tables and views, for both CREATE and ALTER.
            • For some reason, Hive.g was using KW_PROPERTIES; I changed it to KW_TBLPROPERTIES to reduce my own confusion.
            • I had to get rid of one metastore-side validation since it didn't have enough context to discriminate the different forms of ALTER. I think in general it's probably difficult for metastore to validate everything that ql can.
            jvs John Sichi added a comment - Once this is committed, I will update wiki docs accordingly. Notes for reviewers: Now TBLPROPERTIES can be specified for both tables and views, for both CREATE and ALTER. For some reason, Hive.g was using KW_PROPERTIES; I changed it to KW_TBLPROPERTIES to reduce my own confusion. I had to get rid of one metastore-side validation since it didn't have enough context to discriminate the different forms of ALTER. I think in general it's probably difficult for metastore to validate everything that ql can.
            namit Namit Jain added a comment -

            Committed. Thanks John

            namit Namit Jain added a comment - Committed. Thanks John

            People

              jvs John Sichi
              jvs John Sichi
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: