Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24851 )

Change subject: KUDU-3804: Fix symbol hiding for protobuf well-known types
......................................................................


Patch Set 1:

(1 comment)

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

http://gerrit.cloudera.org:8080/#/c/24851/1//COMMIT_MSG@22
PS1, Line 22: This was harmless while the client's bundled protobuf matched the 
host
            : application's protobuf version, but after bumping protobuf to 
3.21.9 for
            : GCC-15 support (KUDU-3803)
> That's fair, some incorrect assumptions went into this. Impala has been usi
While you are at this, did you have a chance to run ./bin/client_symbol-test.sh 
from the Kudu client build directory?  Did it report any issues?

Ideally, the issue of exposing such symbols should have been caught by that 
test.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I64f88bbfab6a709ae31fb18ed97cfc58cb8b647d
Gerrit-Change-Number: 24851
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Tue, 15 Sep 2026 18:05:07 +0000
Gerrit-HasComments: Yes

Reply via email to