Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/19099 )

Change subject: IMPALA-9460: ADD PARTITION doesn't accept SET FORMAT
......................................................................

IMPALA-9460: ADD PARTITION doesn't accept SET FORMAT

This change adds the possibility to set the partition file format
at creation by allowing to use the SET FORMAT clause at the end of
the ALTER TABLE ADD statement. The file format specification will
be applied to all listed partitions, for example:
ALTER TABLE tbl ADD PARTITION (i=1) PARTITION (i=2) SET FILEFORMAT
PARQUET; will create two Parquet-formatted partition metadata
entries.

Tests:
 - analyzer test for the new syntax element
 - e2e test for adding partition in a specified format

Change-Id: I2f78cc3c7eba25383128cd8fd881dd41ddea8b69
Reviewed-on: http://gerrit.cloudera.org:8080/19099
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M fe/src/main/cup/sql-parser.cup
M fe/src/main/java/org/apache/impala/analysis/AlterTableAddPartitionStmt.java
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
M fe/src/test/java/org/apache/impala/analysis/ParserTest.java
M tests/metadata/test_ddl.py
5 files changed, 63 insertions(+), 2 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/19099
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I2f78cc3c7eba25383128cd8fd881dd41ddea8b69
Gerrit-Change-Number: 19099
Gerrit-PatchSet: 8
Gerrit-Owner: Peter Rozsa <[email protected]>
Gerrit-Reviewer: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Peter Rozsa <[email protected]>

Reply via email to