[kudu] branch master updated (786d45d -> e1593f0)

[al...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git.


    from 786d45d  KUDU-2797 p1: expose the tablet's live row count
     new a61d88d  KUDU-2882: Increase the timeout value for TestSentryClientMetrics.Basic
     new e1593f0  KUDU-2897 Making the -mavx2 flag contingent on the architecture rather than the OS alone

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 src/kudu/master/sentry_authz_provider-test.cc | 4 ++--
 thirdparty/build-definitions.sh               | 3 +++
 2 files changed, 5 insertions(+), 2 deletions(-)

[kudu] 01/02: KUDU-2882: Increase the timeout value for TestSentryClientMetrics.Basic

[al...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit a61d88dbc6b9874f1628c0870c6db90104f513e8
Author: helifu <hz...@corp.netease.com>
AuthorDate: Mon Jul 1 18:34:23 2019 +0800

    KUDU-2882: Increase the timeout value for TestSentryClientMetrics.Basic
    
    It will be better to increase the timeout value to avoid the failures
    in some test environments.
    
    Change-Id: I3f7f1dc0887b31c4e1a56071be2f0223aeb292e9
    Reviewed-on: http://gerrit.cloudera.org:8080/13770
    Tested-by: Kudu Jenkins
    Reviewed-by: Andrew Wong <aw...@cloudera.com>
---
 src/kudu/master/sentry_authz_provider-test.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/kudu/master/sentry_authz_provider-test.cc b/src/kudu/master/sentry_authz_provider-test.cc
index fd8230a..2c5efed 100644
--- a/src/kudu/master/sentry_authz_provider-test.cc
+++ b/src/kudu/master/sentry_authz_provider-test.cc
@@ -1393,8 +1393,8 @@ TEST_F(TestSentryClientMetrics, Basic) {
   // Shorten the default timeout parameters: make timeout interval shorter.
   NO_FATALS(sentry_authz_provider_->Stop());
   FLAGS_sentry_service_rpc_addresses = sentry_->address().ToString();
-  FLAGS_sentry_service_send_timeout_seconds = 2;
-  FLAGS_sentry_service_recv_timeout_seconds = 2;
+  FLAGS_sentry_service_send_timeout_seconds = AllowSlowTests() ? 5 : 2;
+  FLAGS_sentry_service_recv_timeout_seconds = AllowSlowTests() ? 5 : 2;
   sentry_authz_provider_.reset(new SentryAuthzProvider(metric_entity_));
   ASSERT_OK(sentry_authz_provider_->Start());
 

[kudu] 02/02: KUDU-2897 Making the -mavx2 flag contingent on the architecture rather than the OS alone

[al...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git

commit e1593f0cf85eeb8ea6e4cb75d610dbff67b89002
Author: iRitwik <ri...@gmail.com>
AuthorDate: Fri Jul 19 13:39:17 2019 -0700

    KUDU-2897 Making the -mavx2 flag contingent on the architecture rather than the OS alone
    
    Earlier, we would use the -mavx2 flag if the OS was UNIX based. However, the flag is only
    usable if the underlying CPU architecture supports it. I have added a check to do that.
    The following commit has been tested on x86 and ARM processors - one supporting AVX2 and
    the other does not. Both were running LINUX.
    
    Change-Id: Ica8c7553cec1fc851685c4a10750e90b3e892500
    Reviewed-on: http://gerrit.cloudera.org:8080/13884
    Reviewed-by: Todd Lipcon <to...@apache.org>
    Tested-by: Kudu Jenkins
    Reviewed-by: Alexey Serbin <as...@cloudera.com>
---
 thirdparty/build-definitions.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/thirdparty/build-definitions.sh b/thirdparty/build-definitions.sh
index ff29e67..b5216cf 100644
--- a/thirdparty/build-definitions.sh
+++ b/thirdparty/build-definitions.sh
@@ -565,6 +565,9 @@ build_bitshuffle() {
   # suffix the AVX2 symbols with '_avx2'. OSX doesn't have objcopy, so we only
   # do this trick on Linux.
   if [ -n "$OS_LINUX" ]; then
+    AVX2_SUPPORT=$(echo | ${CC:-gcc} -mavx2 -dM -E - | awk '$2 == "__AVX2__" { print $3 }')
+  fi
+  if [ -n "$AVX2_SUPPORT" ]; then
     arches="default avx2"
   else
     arches="default"