Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/12495 )
Change subject: IMPALA-8204: checkout to the right commit of Impala-lzo in buildall.sh ...................................................................... Patch Set 2: I still think we should write down the depended Impala-lzo version somewhere. Once I want to build an older version of Impala, I don't need to guess which Impala-lzo commit should I check out to. There's a benifit to do the checkout in buildall.sh. In the future when I want to build an older version (but >2.13, >3.2), I just need to checkout to the right tag in Impala and then run buildall.sh. It'll bootstrap dependencies including the Impala-lzo to the corresponding versions. If you're developing Impala-lzo, you can first commit it locally, and then change the depended commit id in impala-config-branch.sh. I think this is the same with other dependencies. For example, if we want to bump the version of ORC lib, we need to upgrade it in native-toolchain project and then change the corresponding version in impala-config.sh. -- To view, visit http://gerrit.cloudera.org:8080/12495 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: 2.x Gerrit-MessageType: comment Gerrit-Change-Id: I5e4b53d695de04d31d39b6909572907944713ba4 Gerrit-Change-Number: 12495 Gerrit-PatchSet: 2 Gerrit-Owner: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Comment-Date: Sat, 16 Feb 2019 00:21:07 +0000 Gerrit-HasComments: No