Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9257
Change subject: Fix client comaptibility with gcc 4.4 ...................................................................... Fix client comaptibility with gcc 4.4 We expect the client lib to work with gcc 4.4 and C++98. When the int128 type was added this broke compatibility because __int128 support didn’t exist until gcc 4.6. To work around this and still allow gcc 4.6+ and C++11 compilation to support int128, a preprocessor check was added to define KUDU_INT128_SUPPORTED which is used to exclude the incompatible client code. Additionally the int128 insertion operator definitions were moved to their own file to ensure they are not required when working with Kudu’s int128 types. This ensures there aren’t compatibily issues with clients that define their own __int128 insertion operator. Change-Id: I59b40a9718b321df1a5878160ac845d4cf3d9170 --- M src/kudu/client/scan_batch.cc M src/kudu/client/scan_batch.h M src/kudu/client/value-internal.h M src/kudu/client/value.cc M src/kudu/client/value.h M src/kudu/common/column_predicate.cc M src/kudu/common/key_encoder.cc M src/kudu/common/key_encoder.h M src/kudu/common/key_util.cc M src/kudu/common/partial_row.cc M src/kudu/common/partial_row.h M src/kudu/common/types.cc M src/kudu/common/types.h M src/kudu/util/CMakeLists.txt M src/kudu/util/decimal_util.cc M src/kudu/util/decimal_util.h M src/kudu/util/int128-test.cc M src/kudu/util/int128.h R src/kudu/util/int128_util.h 19 files changed, 103 insertions(+), 24 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/57/9257/1 -- To view, visit http://gerrit.cloudera.org:8080/9257 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I59b40a9718b321df1a5878160ac845d4cf3d9170 Gerrit-Change-Number: 9257 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <granthe...@gmail.com>
