Thomas Tauber-Marshall has uploaded a new change for review. http://gerrit.cloudera.org:8080/6902
Change subject: IMPALA-5286: Query fails due to Kudu column name case ...................................................................... IMPALA-5286: Query fails due to Kudu column name case Impala is case insensitive for column names and generally deals with them in all lower case. Kudu is case sensitive. This can lead to a problem when a table is created externally in Kudu with a column name with upper case letters. The particular bug here is that the Analyzer can end up creating two SlotDescriptors that point to the same column because (despite comment in Analyzer.java to the contrary) it does a case sensitive comparison of slot paths to determine if a SlotDescriptor for the path already exists. The fix is to convert slot paths to lower case before doing the comparison. Testing: - Added an e2e test in test_kudu.py. Change-Id: I14aba88510012174716691b9946e1c7d54d01b44 --- M fe/src/main/java/org/apache/impala/analysis/Analyzer.java M tests/query_test/test_kudu.py 2 files changed, 27 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/02/6902/1 -- To view, visit http://gerrit.cloudera.org:8080/6902 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I14aba88510012174716691b9946e1c7d54d01b44 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Thomas Tauber-Marshall <[email protected]>
