Joe McDonnell has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16500 )

Change subject: IMPALA-10198: Unify Java code under the $IMPALA_HOME/java 
directory
......................................................................


Patch Set 3:

(1 comment)

Missed this review comment somehow. I included random other build fixes in this 
upload, and I'm going to split it up into a couple changes to make the pieces 
clear. I'll let you know when this is ready to review.

http://gerrit.cloudera.org:8080/#/c/16500/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/16500/3//COMMIT_MSG@18
PS3, Line 18: This moves all the Java projects other than fe into
> We'll probably have to update some of the wiki pages to reflect this right,
The nice thing about this structure is that the maven commands all work on the 
submodules. Since fe didn't move, the same commands still work from the same 
directory. I will double check that to be sure.



--
To view, visit http://gerrit.cloudera.org:8080/16500
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I08773f4f9d7cb269b0491080078d6e6f490d8d7a
Gerrit-Change-Number: 16500
Gerrit-PatchSet: 3
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Tue, 06 Oct 2020 17:49:15 +0000
Gerrit-HasComments: Yes

Reply via email to