Hello Daniel Becker, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19099
to look at the new patch set (#2).
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
---
M common/thrift/JniCatalog.thrift
M fe/src/main/cup/sql-parser.cup
A
fe/src/main/java/org/apache/impala/analysis/AlterTableAddPartitionSetFileFormatStmt.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
6 files changed, 84 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/99/19099/2
--
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: newpatchset
Gerrit-Change-Id: I2f78cc3c7eba25383128cd8fd881dd41ddea8b69
Gerrit-Change-Number: 19099
Gerrit-PatchSet: 2
Gerrit-Owner: Peter Rozsa <pro...@cloudera.com>
Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
