Quanlong Huang has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9274 )

Change subject: PREVIEW: build ORC C++ lib in toolchain
......................................................................


Patch Set 1:

Hi Tim, one file (include/orc/Statistics.hh) is not added to 
build/orc-1.4.2-p1/include/orc. After manually copying it into the dir, I was 
able to build my orc-patch.

Upgrading from orc-1.2.3 to orc-1.4.2 bring so many benefits. I was able to 
simplify my patch by using the ORC lib to parse the orc file tail, instead of 
writing codes to do this in the initial patch.

BTW, recently orc-1.4.3 was released. We can use the latest version.


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

Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6abd86a5a683f19aa44b47629edf283b938b7e7e
Gerrit-Change-Number: 9274
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Comment-Date: Wed, 14 Feb 2018 10:15:04 +0000
Gerrit-HasComments: No

Reply via email to