[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Lower case struct-field names .. IMPALA-5994: Lower case struct-field names Impala tries to always store column names in lower case. As part of a cleanup of issues related to upper case Kudu column names, a check was added in Analyzer to enforce this. The check fails when doing star expansion on a struct to select all fields in the case where a table was created in Hive with upper case letters in a struct field name. This happens because Hive does not covert struct field names to all lower case in HMS. The solution is to force StructField names to lower case. Testing: - Added a test in test_nested_types.py - Fixed FE test that expected struct field to be output in upper case. Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Reviewed-on: http://gerrit.cloudera.org:8080/8169 Reviewed-by: Thomas Tauber-MarshallTested-by: Impala Public Jenkins --- M fe/src/main/java/org/apache/impala/catalog/StructField.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeDDLTest.java M tests/query_test/test_nested_types.py 3 files changed, 19 insertions(+), 2 deletions(-) Approvals: Thomas Tauber-Marshall: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8169 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Gerrit-Change-Number: 8169 Gerrit-PatchSet: 5 Gerrit-Owner: Thomas Tauber-Marshall Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Thomas Tauber-Marshall
[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Lower case struct-field names .. Patch Set 4: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/8169 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Gerrit-Change-Number: 8169 Gerrit-PatchSet: 4 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Comment-Date: Sat, 30 Sep 2017 01:13:18 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Lower case struct-field names .. Patch Set 4: Code-Review+2 Carrying forward +2. Gvo failed due to trivial test issue. -- To view, visit http://gerrit.cloudera.org:8080/8169 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Gerrit-Change-Number: 8169 Gerrit-PatchSet: 4 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Comment-Date: Fri, 29 Sep 2017 21:14:16 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Lower case struct-field names .. Patch Set 4: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1281/ -- To view, visit http://gerrit.cloudera.org:8080/8169 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Gerrit-Change-Number: 8169 Gerrit-PatchSet: 4 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Comment-Date: Fri, 29 Sep 2017 21:13:11 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names
Hello Lars Volker, Alex Behm, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8169 to look at the new patch set (#4). Change subject: IMPALA-5994: Lower case struct-field names .. IMPALA-5994: Lower case struct-field names Impala tries to always store column names in lower case. As part of a cleanup of issues related to upper case Kudu column names, a check was added in Analyzer to enforce this. The check fails when doing star expansion on a struct to select all fields in the case where a table was created in Hive with upper case letters in a struct field name. This happens because Hive does not covert struct field names to all lower case in HMS. The solution is to force StructField names to lower case. Testing: - Added a test in test_nested_types.py - Fixed FE test that expected struct field to be output in upper case. Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e --- M fe/src/main/java/org/apache/impala/catalog/StructField.java M fe/src/test/java/org/apache/impala/analysis/AnalyzeDDLTest.java M tests/query_test/test_nested_types.py 3 files changed, 19 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/69/8169/4 -- To view, visit http://gerrit.cloudera.org:8080/8169 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Gerrit-Change-Number: 8169 Gerrit-PatchSet: 4 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Thomas Tauber-Marshall
[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Lower case struct-field names .. Patch Set 3: Verified-1 Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/1277/ -- To view, visit http://gerrit.cloudera.org:8080/8169 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Gerrit-Change-Number: 8169 Gerrit-PatchSet: 3 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Comment-Date: Fri, 29 Sep 2017 20:53:25 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names
Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Lower case struct-field names .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8169 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Gerrit-Change-Number: 8169 Gerrit-PatchSet: 3 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Comment-Date: Fri, 29 Sep 2017 16:21:57 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Lower case struct-field names .. Patch Set 3: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/1277/ -- To view, visit http://gerrit.cloudera.org:8080/8169 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Gerrit-Change-Number: 8169 Gerrit-PatchSet: 3 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Comment-Date: Fri, 29 Sep 2017 16:16:42 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names
Thomas Tauber-Marshall has posted comments on this change. ( http://gerrit.cloudera.org:8080/8169 ) Change subject: IMPALA-5994: Lower case struct-field names .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/8169/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/8169/1//COMMIT_MSG@7 PS1, Line 7: IMPALA-5994: Lower case struct-field names > Commit msg should describe the change/fix, not the symptom of the bug. For Done http://gerrit.cloudera.org:8080/#/c/8169/2/fe/src/main/java/org/apache/impala/catalog/StructField.java File fe/src/main/java/org/apache/impala/catalog/StructField.java: http://gerrit.cloudera.org:8080/#/c/8169/2/fe/src/main/java/org/apache/impala/catalog/StructField.java@37 PS2, Line 37: // Impala expects field names to be in lower case, but type strings stored in the HMS > // Impala expects field names to be in lower case, but type strings stored Done -- To view, visit http://gerrit.cloudera.org:8080/8169 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Gerrit-Change-Number: 8169 Gerrit-PatchSet: 3 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Thomas Tauber-Marshall Gerrit-Comment-Date: Fri, 29 Sep 2017 16:16:22 + Gerrit-HasComments: Yes
[Impala-ASF-CR] IMPALA-5994: Lower case struct-field names
Hello Lars Volker, Alex Behm, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8169 to look at the new patch set (#3). Change subject: IMPALA-5994: Lower case struct-field names .. IMPALA-5994: Lower case struct-field names Impala tries to always store column names in lower case. As part of a cleanup of issues related to upper case Kudu column names, a check was added in Analyzer to enforce this. The check fails when doing star expansion on a struct to select all fields in the case where a table was created in Hive with upper case letters in a struct field name. This happens because Hive does not covert struct field names to all lower case in HMS. The solution is to force StructField names to lower case. Testing: - Added a test in test_nested_types.py Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e --- M fe/src/main/java/org/apache/impala/catalog/StructField.java M tests/query_test/test_nested_types.py 2 files changed, 18 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/69/8169/3 -- To view, visit http://gerrit.cloudera.org:8080/8169 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Iacd9714ac2301a55ee8b64f0102f6f156fb0370e Gerrit-Change-Number: 8169 Gerrit-PatchSet: 3 Gerrit-Owner: Thomas Tauber-MarshallGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Thomas Tauber-Marshall