You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by gr...@apache.org on 2020/04/27 00:43:42 UTC

[kudu] branch master updated (8d4853f -> ae4b715)

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

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


    from 8d4853f  clients: prefer tservers in the same location when multiple are local
     new cc0ee37  [docker] Clarify how to run Impala in standalone
     new ae4b715  [binary-jar] Fix macOS binary jar libc++abi error

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:
 build-support/mini-cluster/LICENSE-BINARY-JAR-OSX.txt            | 4 ++++
 build-support/mini-cluster/relocate_binaries_for_mini_cluster.py | 2 +-
 examples/quickstart/impala/README.adoc                           | 4 ++--
 3 files changed, 7 insertions(+), 3 deletions(-)


[kudu] 01/02: [docker] Clarify how to run Impala in standalone

Posted by gr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit cc0ee37b6b82918897bb2ed4786c32aeda1711ee
Author: Romain Rigaux <ro...@gmail.com>
AuthorDate: Thu Apr 23 11:28:18 2020 -0700

    [docker] Clarify how to run Impala in standalone
    
    Change-Id: Ib5feec357f554b72856a39d0209c60b91d9816e0
    Reviewed-on: http://gerrit.cloudera.org:8080/15810
    Tested-by: Kudu Jenkins
    Reviewed-by: Attila Bukor <ab...@apache.org>
    Reviewed-by: Grant Henke <gr...@apache.org>
---
 examples/quickstart/impala/README.adoc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/examples/quickstart/impala/README.adoc b/examples/quickstart/impala/README.adoc
index c0f7644..03994ee 100644
--- a/examples/quickstart/impala/README.adoc
+++ b/examples/quickstart/impala/README.adoc
@@ -43,8 +43,8 @@ docker run -d --name kudu-impala --network="docker_default" \
 You can view the running Impala instance at link:http://localhost:25000[localhost:25000]
 once it is up and running. It may take a few seconds to start.
 
-NOTE: `--network="docker_default"` is specified to connect the container the
-same network as the quickstart cluster.
+NOTE: `--network="docker_default"` is optional and used for enabling Impala to connect to the
+same network as the Kudu quickstart cluster hence allowing the use of Kudu tables.
 
 NOTE: You can remove the `-d` flag to run the container in the foreground.
 


[kudu] 02/02: [binary-jar] Fix macOS binary jar libc++abi error

Posted by gr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit ae4b71548726314a0bc0a5258de454d1a65d08a3
Author: Grant Henke <gr...@apache.org>
AuthorDate: Sun Apr 26 14:29:16 2020 -0500

    [binary-jar] Fix macOS binary jar libc++abi error
    
    The macOS binary jar was broken by commit 458e6ce because
    thirdparty now builds with libc++abi for normal builds.
    
    This resulted in an error like the below when using a macOS binary jar:
        [main] INFO org.apache.kudu.shaded.com.google.gradle.osdetector.OsDetector - os.detected.classifier=osx-x86_64
        [main] INFO org.apache.kudu.test.cluster.KuduBinaryLocator - Using Kudu binary jar directory: /var/folders/3y/nrs6__c10n107f357n7yx2100000gp/T/kudu-binary-jar6249991106299083358
        [main] INFO org.apache.kudu.test.cluster.MiniKuduCluster - Starting process: [/var/folders/3y/nrs6__c10n107f357n7yx2100000gp/T/kudu-binary-jar6249991106299083358/kudu-binary-1.12.0-osx-x86_64/bin/kudu, test, mini_cluster, --serialization=pb]
        [cluster stderr printer] INFO org.apache.kudu.test.cluster.MiniKuduCluster - dyld: Library not loaded: @rpath/libc++abi.1.dylib
        [cluster stderr printer] INFO org.apache.kudu.test.cluster.MiniKuduCluster -   Referenced from: /private/var/folders/3y/nrs6__c10n107f357n7yx2100000gp/T/kudu-binary-jar6249991106299083358/kudu-binary-1.12.0-osx-x86_64/lib/libglog.0.dylib
        [cluster stderr printer] INFO org.apache.kudu.test.cluster.MiniKuduCluster -   Reason: image not found
        [main] ERROR org.apache.kudu.test.junit.RetryRule - org.apache.kudu.examples.ExampleTest.testCreateExampleTable: failed attempt 1
        java.io.EOFException
    
    This patch fixes the issue by adjusting the `PAT_MACOS_LIB_EXCLUDE`
    regex to ensure the libc++abi library is not excluded from the jar.
    Additionaly it adjusts the licence file to include the libc++abi.
    
    Change-Id: I395fd39d62fd7bfcd2aa2b0781d77c3a16e19d4b
    Reviewed-on: http://gerrit.cloudera.org:8080/15812
    Tested-by: Kudu Jenkins
    Reviewed-by: Hao Hao <ha...@cloudera.com>
---
 build-support/mini-cluster/LICENSE-BINARY-JAR-OSX.txt            | 4 ++++
 build-support/mini-cluster/relocate_binaries_for_mini_cluster.py | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/build-support/mini-cluster/LICENSE-BINARY-JAR-OSX.txt b/build-support/mini-cluster/LICENSE-BINARY-JAR-OSX.txt
index 46c17f6..3da6156 100644
--- a/build-support/mini-cluster/LICENSE-BINARY-JAR-OSX.txt
+++ b/build-support/mini-cluster/LICENSE-BINARY-JAR-OSX.txt
@@ -10,3 +10,7 @@ OpenSSL: OpenSSL software license
 libraries: libssl, libcrypto
 Source: Homebrew repository
 
+--------------------------------------------------------------------------------
+LLVM: Apache 2.0 License
+libraries: libc\+\+abi
+Source: LLVM source tarball
diff --git a/build-support/mini-cluster/relocate_binaries_for_mini_cluster.py b/build-support/mini-cluster/relocate_binaries_for_mini_cluster.py
index f3c0aad..c17e911 100755
--- a/build-support/mini-cluster/relocate_binaries_for_mini_cluster.py
+++ b/build-support/mini-cluster/relocate_binaries_for_mini_cluster.py
@@ -91,7 +91,7 @@ PAT_MACOS_LIB_EXCLUDE = re.compile(r"""(AppleFSCompression$|
                                         libSystem|
                                         libapple_nghttp2|
                                         libarchive|
-                                        libc\+\+|
+                                        libc\+\+\.|
                                         libenergytrace|
                                         libicucore|
                                         libncurses|