Skip to content
This repository was archived by the owner on Mar 3, 2023. It is now read-only.

Commit 12b8abc

Browse files
authored
Updates to fix issues with the Heron API dependencies (#3844)
1 parent b4d640f commit 12b8abc

File tree

6 files changed

+22
-11
lines changed

6 files changed

+22
-11
lines changed

WORKSPACE

+1
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,7 @@ maven_install(
124124
"tech.tablesaw:tablesaw-core:0.11.4",
125125
"org.glassfish.hk2.external:aopalliance-repackaged:2.5.0-b32",
126126
"org.apache.commons:commons-compress:1.14",
127+
"org.apache.commons:commons-lang3:3.12.0",
127128
"commons-io:commons-io:2.4",
128129
"commons-collections:commons-collections:3.2.1",
129130
"commons-cli:commons-cli:1.3.1",

heron/api/src/java/shade.conf

+4-4
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
# specific language governing permissions and limitations
1616
# under the License.
1717

18-
rule com.google.protobuf** org.apache.heron.shaded.@0
19-
rule org.slf4j** org.apache.heron.shaded.@0
20-
rule javax.xml.bind** org.apache.heron.shaded.@0
21-
rule org.apache.commons** org.apache.heron.shaded.@0
18+
zap com.google.protobuf**
19+
zap org.slf4j**
20+
zap javax.xml.bind**
21+
zap org.apache.commons.lang3**

maven_install.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"dependency_tree": {
33
"__AUTOGENERATED_FILE_DO_NOT_MODIFY_THIS_FILE_MANUALLY": "THERE_IS_NO_DATA_ONLY_ZUUL",
4-
"__INPUT_ARTIFACTS_HASH": 389120310,
4+
"__INPUT_ARTIFACTS_HASH": 1411695818,
55
"__RESOLVED_ARTIFACTS_HASH": 1590208671,
66
"conflict_resolution": {},
77
"dependencies": [

release/maven/maven-pom-version.sh

+13-3
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,19 @@ cat ./maven/heron-no-kryo.template.pom | \
3434
<version>3.12.0</version>\
3535
</dependency>\
3636
<dependency>\
37-
<groupId>org.glassfish.jersey.media</groupId>\
38-
<artifactId>jersey-media-jaxb</artifactId>\
39-
<version>2.25.1</version>\
37+
<groupId>com.google.protobuf</groupId>\
38+
<artifactId>protobuf-java</artifactId>\
39+
<version>3.16.1</version>\
40+
</dependency>\
41+
<dependency>\
42+
<groupId>javax.xml.bind</groupId>\
43+
<artifactId>jaxb-api</artifactId>\
44+
<version>2.3.0</version>\
45+
</dependency>\
46+
<dependency>\
47+
<groupId>org.slf4j</groupId>\
48+
<artifactId>jul-to-slf4j</artifactId>\
49+
<version>1.7.36</version>\
4050
</dependency>", $0); print}' | \
4151
xmllint --format - \
4252
>> ./heron-api-$1.pom

scripts/ci/build_maven_artifacts.sh

100644100755
+1-1
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ mkdir -p $OUTPUT_DIR/$VERSION_TAG/heron-kafka-bolt
6969

7070
echo "Copy heron-api artifacts ... "
7171
cp -p -f ./release/heron-api-$VERSION_TAG.pom $OUTPUT_DIR/$VERSION_TAG/heron-api/
72-
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
72+
cp -p -f ./bazel-bin/heron/api/src/java/api-shaded.jar $OUTPUT_DIR/$VERSION_TAG/heron-api/heron-api-$VERSION_TAG.jar
7373
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
7474
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
7575

tools/rules/heron_deps.bzl

+2-2
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@ def heron_java_proto_files():
3131
"//heron/proto:proto_topology_java",
3232
"//heron/proto:proto_tuple_java",
3333
"//heron/proto:proto_stmgr_java",
34-
"@com_google_protobuf//:protobuf_java",
34+
"@maven//:com_google_protobuf_protobuf_java",
3535
]
3636

3737
def heron_java_api_proto_files():
3838
return [
3939
"//heron/proto:proto_topology_java",
40-
"@com_google_protobuf//:protobuf_java",
40+
"@maven//:com_google_protobuf_protobuf_java",
4141
]

0 commit comments

Comments
 (0)