kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a...@apache.org
Subject [2/3] kudu git commit: C++ client: hide deflate symbols
Date Wed, 09 Nov 2016 23:49:33 GMT
C++ client: hide deflate symbols

These snuck in via commit 8a4bb33. Oddly, client_symbol-test.sh was only
observed to be failing on el6 (not on Ubuntu 14.04 or 16.04). The running
theory is it's because the devtoolset linker is old enough that it can't
figure out that these symbols aren't callable by any client code path.

Change-Id: Ibb5b6cf186a9a18583c4695b134c0729b81a4c01
Reviewed-on: http://gerrit.cloudera.org:8080/5023
Reviewed-by: Todd Lipcon <todd@apache.org>
Tested-by: Kudu Jenkins


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/388908c0
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/388908c0
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/388908c0

Branch: refs/heads/master
Commit: 388908c0d5f2fc810a92ccc3ab0adca17f79358e
Parents: dbac80f
Author: Adar Dembo <adar@cloudera.com>
Authored: Wed Nov 9 14:49:50 2016 -0800
Committer: Adar Dembo <adar@cloudera.com>
Committed: Wed Nov 9 23:48:55 2016 +0000

----------------------------------------------------------------------
 src/kudu/client/symbols.map | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/388908c0/src/kudu/client/symbols.map
----------------------------------------------------------------------
diff --git a/src/kudu/client/symbols.map b/src/kudu/client/symbols.map
index 7c6db4d..fe11596 100644
--- a/src/kudu/client/symbols.map
+++ b/src/kudu/client/symbols.map
@@ -14,6 +14,7 @@
     # zlib
     adler32*;
     crc32*;
+    deflate*;
     get_crc_table;
     inflate*;
     zError;


Mime
View raw message