You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@heron.apache.org by ni...@apache.org on 2022/07/09 17:48:15 UTC

[incubator-heron] branch master updated: Updates to fix issues with the Heron API dependencies (#3844)

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

nicknezis pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-heron.git


The following commit(s) were added to refs/heads/master by this push:
     new 12b8abc152f Updates to fix issues with the Heron API dependencies (#3844)
12b8abc152f is described below

commit 12b8abc152f7a23e87ef19299ff92423a6ee1469
Author: Nicholas Nezis <ni...@gmail.com>
AuthorDate: Sat Jul 9 13:48:08 2022 -0400

    Updates to fix issues with the Heron API dependencies (#3844)
---
 WORKSPACE                           |  1 +
 heron/api/src/java/shade.conf       |  8 ++++----
 maven_install.json                  |  2 +-
 release/maven/maven-pom-version.sh  | 16 +++++++++++++---
 scripts/ci/build_maven_artifacts.sh |  2 +-
 tools/rules/heron_deps.bzl          |  4 ++--
 6 files changed, 22 insertions(+), 11 deletions(-)

diff --git a/WORKSPACE b/WORKSPACE
index fe7b4c52579..ec7c59202de 100644
--- a/WORKSPACE
+++ b/WORKSPACE
@@ -124,6 +124,7 @@ maven_install(
         "tech.tablesaw:tablesaw-core:0.11.4",
         "org.glassfish.hk2.external:aopalliance-repackaged:2.5.0-b32",
         "org.apache.commons:commons-compress:1.14",
+        "org.apache.commons:commons-lang3:3.12.0",
         "commons-io:commons-io:2.4",
         "commons-collections:commons-collections:3.2.1",
         "commons-cli:commons-cli:1.3.1",
diff --git a/heron/api/src/java/shade.conf b/heron/api/src/java/shade.conf
index ebf96076a15..f95810af16c 100644
--- a/heron/api/src/java/shade.conf
+++ b/heron/api/src/java/shade.conf
@@ -15,7 +15,7 @@
 #  specific language governing permissions and limitations
 #  under the License.
 
-rule com.google.protobuf** org.apache.heron.shaded.@0
-rule org.slf4j** org.apache.heron.shaded.@0
-rule javax.xml.bind** org.apache.heron.shaded.@0
-rule org.apache.commons** org.apache.heron.shaded.@0
+zap com.google.protobuf**
+zap org.slf4j**
+zap javax.xml.bind**
+zap org.apache.commons.lang3**
diff --git a/maven_install.json b/maven_install.json
index ebe4f527a2c..fc7a60733ce 100644
--- a/maven_install.json
+++ b/maven_install.json
@@ -1,7 +1,7 @@
 {
     "dependency_tree": {
         "__AUTOGENERATED_FILE_DO_NOT_MODIFY_THIS_FILE_MANUALLY": "THERE_IS_NO_DATA_ONLY_ZUUL",
-        "__INPUT_ARTIFACTS_HASH": 389120310,
+        "__INPUT_ARTIFACTS_HASH": 1411695818,
         "__RESOLVED_ARTIFACTS_HASH": 1590208671,
         "conflict_resolution": {},
         "dependencies": [
diff --git a/release/maven/maven-pom-version.sh b/release/maven/maven-pom-version.sh
index 92580464449..21cb43f1e49 100755
--- a/release/maven/maven-pom-version.sh
+++ b/release/maven/maven-pom-version.sh
@@ -34,9 +34,19 @@ cat ./maven/heron-no-kryo.template.pom | \
                             <version>3.12.0</version>\
                         </dependency>\
                         <dependency>\
-                            <groupId>org.glassfish.jersey.media</groupId>\
-                             <artifactId>jersey-media-jaxb</artifactId>\
-                              <version>2.25.1</version>\
+                          <groupId>com.google.protobuf</groupId>\
+                            <artifactId>protobuf-java</artifactId>\
+                            <version>3.16.1</version>\
+                        </dependency>\
+                        <dependency>\
+                          <groupId>javax.xml.bind</groupId>\
+                            <artifactId>jaxb-api</artifactId>\
+                            <version>2.3.0</version>\
+                        </dependency>\
+                        <dependency>\
+                          <groupId>org.slf4j</groupId>\
+                            <artifactId>jul-to-slf4j</artifactId>\
+                            <version>1.7.36</version>\
                         </dependency>", $0); print}' | \
     xmllint --format - \
     >> ./heron-api-$1.pom
diff --git a/scripts/ci/build_maven_artifacts.sh b/scripts/ci/build_maven_artifacts.sh
old mode 100644
new mode 100755
index 6b89ba6a308..f9fda6f8a4f
--- a/scripts/ci/build_maven_artifacts.sh
+++ b/scripts/ci/build_maven_artifacts.sh
@@ -69,7 +69,7 @@ mkdir -p $OUTPUT_DIR/$VERSION_TAG/heron-kafka-bolt
 
 echo "Copy heron-api artifacts ... "
 cp -p -f ./release/heron-api-$VERSION_TAG.pom $OUTPUT_DIR/$VERSION_TAG/heron-api/
-cp -p -f ./bazel-bin/heron/api/src/java/api-java-low-level-functional.jar $OUTPUT_DIR/$VERSION_TAG/heron-api/heron-api-$VERSION_TAG.jar
+cp -p -f ./bazel-bin/heron/api/src/java/api-shaded.jar $OUTPUT_DIR/$VERSION_TAG/heron-api/heron-api-$VERSION_TAG.jar
 cp -p -f ./bazel-bin/heron/api/src/java/heron-api-javadoc.zip $OUTPUT_DIR/$VERSION_TAG/heron-api/heron-api-$VERSION_TAG-javadoc.jar
 cp -p -f ./bazel-bin/heron/api/src/java/libapi-java-low-level-functional-src.jar $OUTPUT_DIR/$VERSION_TAG/heron-api/heron-api-$VERSION_TAG-sources.jar
 
diff --git a/tools/rules/heron_deps.bzl b/tools/rules/heron_deps.bzl
index c51942c1a08..63b1524eccc 100644
--- a/tools/rules/heron_deps.bzl
+++ b/tools/rules/heron_deps.bzl
@@ -31,11 +31,11 @@ def heron_java_proto_files():
         "//heron/proto:proto_topology_java",
         "//heron/proto:proto_tuple_java",
         "//heron/proto:proto_stmgr_java",
-        "@com_google_protobuf//:protobuf_java",
+        "@maven//:com_google_protobuf_protobuf_java",
     ]
 
 def heron_java_api_proto_files():
     return [
         "//heron/proto:proto_topology_java",
-        "@com_google_protobuf//:protobuf_java",
+        "@maven//:com_google_protobuf_protobuf_java",
     ]