Hello Joe McDonnell,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19268
to look at the new patch set (#3).
Change subject: IMPALA-9487: Add erasure coding policy to SHOW, DESCRIBE
......................................................................
IMPALA-9487: Add erasure coding policy to SHOW, DESCRIBE
Adds erasure coding policy to introspection commands SHOW FILES, SHOW
PARTITIONS, SHOW TABLE STATS, and DESCRIBE EXTENDED.
Remove `throws IOException` for methods that don't throw. Removes null
check for getSd because getStorageDescriptorInfo - which is called right
after getTableMetaDataInformation - uses it without checking for null.
Testing:
- ran backend, end-to-end, and custom cluster tests with erasure coding
- ran backend, end-to-end, and custom cluster tests with exhaustive
strategy
Change-Id: Idd95f2d18b3980581788c92993b6d2f53504b5e0
---
M fe/src/main/java/org/apache/impala/catalog/FeFsTable.java
M fe/src/main/java/org/apache/impala/catalog/FeIcebergTable.java
M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java
M fe/src/main/java/org/apache/impala/common/FileSystemUtil.java
M fe/src/main/java/org/apache/impala/util/HiveMetadataFormatUtils.java
M testdata/workloads/functional-query/queries/QueryTest/acid-compaction.test
M testdata/workloads/functional-query/queries/QueryTest/acid-compute-stats.test
M testdata/workloads/functional-query/queries/QueryTest/acid-insert.test
M testdata/workloads/functional-query/queries/QueryTest/acid-truncate.test
M testdata/workloads/functional-query/queries/QueryTest/acid.test
M
testdata/workloads/functional-query/queries/QueryTest/alter-table-hdfs-caching.test
M testdata/workloads/functional-query/queries/QueryTest/alter-table.test
M
testdata/workloads/functional-query/queries/QueryTest/compute-stats-avro-catalog-v2.test
M testdata/workloads/functional-query/queries/QueryTest/compute-stats-avro.test
M testdata/workloads/functional-query/queries/QueryTest/compute-stats-date.test
M
testdata/workloads/functional-query/queries/QueryTest/compute-stats-decimal.test
M
testdata/workloads/functional-query/queries/QueryTest/compute-stats-incremental.test
M
testdata/workloads/functional-query/queries/QueryTest/compute-stats-many-partitions.test
M testdata/workloads/functional-query/queries/QueryTest/compute-stats.test
M testdata/workloads/functional-query/queries/QueryTest/corrupt-stats.test
M
testdata/workloads/functional-query/queries/QueryTest/create-table-like-table.test
M testdata/workloads/functional-query/queries/QueryTest/full-acid-scans.test
M testdata/workloads/functional-query/queries/QueryTest/grant_revoke.test
M
testdata/workloads/functional-query/queries/QueryTest/hdfs-caching-validation.test
M testdata/workloads/functional-query/queries/QueryTest/hdfs-caching.test
M testdata/workloads/functional-query/queries/QueryTest/hidden-files.test
M testdata/workloads/functional-query/queries/QueryTest/iceberg-catalogs.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-compound-predicate-push-down.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-compute-stats.test
M testdata/workloads/functional-query/queries/QueryTest/iceberg-create.test
M testdata/workloads/functional-query/queries/QueryTest/iceberg-ctas.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-in-predicate-push-down.test
M testdata/workloads/functional-query/queries/QueryTest/iceberg-insert.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-is-null-predicate-push-down.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-multiple-storage-locations-table.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-partition-transform-insert.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-partitioned-insert.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-plain-count-star-optimization.test
M testdata/workloads/functional-query/queries/QueryTest/iceberg-query.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-upper-lower-bound-metrics.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-v2-read-position-deletes-orc.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-v2-read-position-deletes.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-write-many-files-stress.test
M
testdata/workloads/functional-query/queries/QueryTest/iceberg-write-many-files.test
M
testdata/workloads/functional-query/queries/QueryTest/multiple-filesystems.test
M
testdata/workloads/functional-query/queries/QueryTest/partition-ddl-predicates-all-fs.test
M testdata/workloads/functional-query/queries/QueryTest/show-stats.test
M testdata/workloads/functional-query/queries/QueryTest/show.test
M testdata/workloads/functional-query/queries/QueryTest/stats-extrapolation.test
M testdata/workloads/functional-query/queries/QueryTest/truncate-table.test
M tests/common/impala_test_suite.py
M tests/metadata/test_hms_integration.py
52 files changed, 4,153 insertions(+), 4,135 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/68/19268/3
--
To view, visit http://gerrit.cloudera.org:8080/19268
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idd95f2d18b3980581788c92993b6d2f53504b5e0
Gerrit-Change-Number: 19268
Gerrit-PatchSet: 3
Gerrit-Owner: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
