Hello Alexey Serbin, Ashwani Raina, Kudu Jenkins, Abhishek Chennaka,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/24851

to look at the new patch set (#2).

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

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

The exported client library's linker version script (symbols.map) hides
glog, gflags, and protobuf symbols behind a "*google::*" C++ demangled
wildcard so they don't collide with a host application's own copies of
these libraries.

protobuf's generated descriptor tables for its well-known types
(any.proto, descriptor.proto, source_context.proto, type.proto,
wrappers.proto) are emitted as plain, non-namespaced globals (e.g.
descriptor_table_google_2fprotobuf_2fdescriptor_2eproto) so they are not
matched by "*google::*" and leak out of the DSO with default visibility.

This causes conflicts in Impala which crash while loading
libkudu_client. Mask them out as well.

Change-Id: I64f88bbfab6a709ae31fb18ed97cfc58cb8b647d
Assisted-By: Claude Sonnet 5 (GitHub Copilot)
---
M src/kudu/client/symbols.map
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/51/24851/2
--
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: newpatchset
Gerrit-Change-Id: I64f88bbfab6a709ae31fb18ed97cfc58cb8b647d
Gerrit-Change-Number: 24851
Gerrit-PatchSet: 2
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]>

Reply via email to