Tamas Mate has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20156 )

Change subject: IMPALA-12263: Build with C++ Avro library when USE_AVRO_CPP is 
true
......................................................................


Patch Set 1:

(1 comment)

Thank you for the review Daniel!

http://gerrit.cloudera.org:8080/#/c/20156/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/20156/1//COMMIT_MSG@13
PS1, Line 13: Building with the C++ library fails at this point.
> This is not going to be fixed in this commit, right?
It is not going to be part of this commit. I am trying to implement it 
incrementally, so the review cycles can be short.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0a81c3f7ab5a6651d507d8d9fac77ea17b8bb1a1
Gerrit-Change-Number: 20156
Gerrit-PatchSet: 1
Gerrit-Owner: Tamas Mate <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]>
Gerrit-Reviewer: Tamas Mate <[email protected]>
Gerrit-Comment-Date: Mon, 03 Jul 2023 15:27:55 +0000
Gerrit-HasComments: Yes

Reply via email to