Lars Volker has posted comments on this change. Change subject: IMPALA-4047: DO NOT SUBMIT Highlight all occurrences of CDH/cdh ......................................................................
Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/4187/1/bin/bootstrap_toolchain.py File bin/bootstrap_toolchain.py: > I think it deserves it's own issue, probably. Created IMPALA-4083 http://gerrit.cloudera.org:8080/#/c/4187/1/fe/pom.xml File fe/pom.xml: Line 198 > Yeah, but why CZT? Are they bound up together, somehow? I looked around and from what I found here (http://czt.sourceforge.net/dev/java-cup/) they say that java-cup wasn't in active development anymore, so czt forked it. However the cup website (http://www2.cs.tum.edu/projects/cup/) has downloads from June 2016. IMPALA-3418 introduced this, so Alex might have more information. I opened IMPALA-4084 to track renaming them. http://gerrit.cloudera.org:8080/#/c/4187/1/tests/comparison/leopard/impala_docker_env.py File tests/comparison/leopard/impala_docker_env.py: PS1, Line 34: : > 1. If there's nothing wrong with Cloudera producing an Impala Docker image, Ok, I will keep this as origin/cdh5-trunk then. I created IMPALA-4085 to track changing the name. Mike, do you want to take this one, too? -- To view, visit http://gerrit.cloudera.org:8080/4187 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Icb37e2ef0cd9fa0e581d359c5dd3db7812b7b2c8 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Lars Volker <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Jim Apple <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-HasComments: Yes
