You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Wenzhe Zhou (Code Review)" <ge...@cloudera.org> on 2022/07/20 22:22:48 UTC

[Impala-ASF-CR] IMPALA-11445: Fix bug in firing insert event of partitions located in different FS

Wenzhe Zhou has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/18759 )

Change subject: IMPALA-11445: Fix bug in firing insert event of partitions located in different FS
......................................................................

IMPALA-11445: Fix bug in firing insert event of partitions located in different FS

When adding a partition with location in a file system which is
different from the file system of the table location, Impala accept
it. But when insert values to the table, catalogd throw exception.

This patch fix the issue by using the right FileSystem object.

Testing:
 - Added new test case with partitions on different file systems.
   Ran the test on S3.
 - Did manual tests in cluster with partitions on HDFS and Ozone.
 - Passed core test.

Change-Id: I0491ee1bf40c3d5240f9124cef3f3169c44a8267
---
M fe/src/main/java/org/apache/impala/catalog/FeFsTable.java
M fe/src/main/java/org/apache/impala/service/CatalogOpExecutor.java
A testdata/workloads/functional-query/queries/QueryTest/partition-location.test
M tests/query_test/test_partitioning.py
4 files changed, 120 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/59/18759/2
-- 
To view, visit http://gerrit.cloudera.org:8080/18759
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0491ee1bf40c3d5240f9124cef3f3169c44a8267
Gerrit-Change-Number: 18759
Gerrit-PatchSet: 2
Gerrit-Owner: Wenzhe Zhou <wz...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <hu...@gmail.com>
Gerrit-Reviewer: Wenzhe Zhou <wz...@cloudera.com>