Tags: charygao/kudu
Tags
[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 <[email protected]> (cherry picked from commit ae4b715) Reviewed-on: http://gerrit.cloudera.org:8080/15813 Tested-by: Grant Henke <[email protected]> Reviewed-by: Alexey Serbin <[email protected]>
mini-cluster: exclude libpcre for license check While building mini cluster binaries on centos7, I saw error message 'unknown license: lib/libpcre.so.1' during license check on artifact. By running lddtree on the binary, I found out libpcre is introduced as transitive dependency libcurl.so.4 => /root/kudu/thirdparty/installed/uninstrumented/lib/libcurl.so.4 libk5crypto.so.3 => /lib64/libk5crypto.so.3 libkrb5support.so.0 => /lib64/libkrb5support.so.0 libselinux.so.1 => /lib64/libselinux.so.1 libpcre.so.1 => /lib64/libpcre.so.1 This patch excludes 'libpcre' for license check to account for licensing information only relevant to the binary artifacts. Change-Id: I0fc69eaf57025b1ce2f7857808e60218dc4f2771 Reviewed-on: http://gerrit.cloudera.org:8080/15738 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> (cherry picked from commit cd95fef) Reviewed-on: http://gerrit.cloudera.org:8080/15744 Tested-by: Hao Hao <[email protected]>
[mini-cluster] fix on building mini-cluster JAR Prior to this patch, build_mini_cluster_binaries.sh script didn't properly handle the case of running in the same workspace that was used to build the mini-cluster JAR for prior releases. That might result in not building the required JAR, so no binary content would be uploaded by running publish_mini_cluster_binaries.sh script later on. This patch fixes the issue: by default, the build directory is re-created upon every run of the build_mini_cluster_binaries.sh script unless MINI_CLUSTER_NO_FRESH_BUILD environment variable is set. Also, even if MINI_CLUSTER_NO_FRESH_BUILD is set and the build directory is not re-created, the script now warns on ambiguity when multiple artifact directories are present. Change-Id: Ic2d5c2a83e2a0ab959e2eb7898d47ab736cd1e1f Reviewed-on: http://gerrit.cloudera.org:8080/14702 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> (cherry picked from commit 6406123) Reviewed-on: http://gerrit.cloudera.org:8080/14728 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]>
[java] fixed bug in the connection negotiation code This patch fixes a typo in the connection negotiation code in the Java client. Prior to this fix, channel binding information was not verified during connection negotiation because the peer certificate was not set. In addition, I modified the error handing code in Negotiator.java to abort connection negotiation upon receiving SSLPeerUnverifiedException due to the absence of the channel binding information or the presence of the invalid one. I also added a test to verify that Kudu Java client doesn't connect to a Kudu server which doesn't provide valid channel binding information during the connection negotiation phase. Kudos to Andy Singer for pointing to the bug. Change-Id: I7bfd428128e224f03901a6cd7b33283495a28d54 Reviewed-on: http://gerrit.cloudera.org:8080/14713 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> Reviewed-by: Todd Lipcon <[email protected]> (cherry picked from commit a0e8964) Reviewed-on: http://gerrit.cloudera.org:8080/14724 Reviewed-by: Alexey Serbin <[email protected]>
[docs] update info on KUDU-2990 in known issues Kudu 1.10.1 and 1.11.1 patch releases are in the making, so let's update the wording in known issues section to prepare the document for publishing at the project's web site. Change-Id: If9678cbd98c1d82a84ddf8ada8caba8746a205bd Reviewed-on: http://gerrit.cloudera.org:8080/14698 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Alexey Serbin <[email protected]> (cherry picked from commit b3147d8) Reviewed-on: http://gerrit.cloudera.org:8080/14699 Reviewed-by: Alexey Serbin <[email protected]>
[build-support] fix on enable_devtoolset_inner.sh When ccache is not present on the build machine, the prior version of this script output an error: enable_devtoolset_inner.sh: line 28: [: too many arguments This patch fixes this minor issue. Change-Id: I49b1db9933371304674b148cc4d71c3703be1772 Reviewed-on: http://gerrit.cloudera.org:8080/14595 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> Reviewed-by: Andrew Wong <[email protected]> (cherry picked from commit 719418a) Reviewed-on: http://gerrit.cloudera.org:8080/14682 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Alexey Serbin <[email protected]>
[examples] update version to 1.10.1 Change-Id: I7214586c4fefb260b9852abbc1533b6e8e7f2cf6 Reviewed-on: http://gerrit.cloudera.org:8080/14674 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Alexey Serbin <[email protected]>
PreviousNext