Skip to content

Commit 656d3ed

Browse files
authored
Remove flink streaming scala dependency as we no longer need it (#327)
## Summary Remove flink streaming scala dependency as we no longer need it otherwise we will run into runtime error saying flink-shaded-guava package not found ## Checklist - [ ] Added Unit Tests - [x] Covered by existing CI - [ ] Integration tested - [ ] Documentation update <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Bug Fixes** - Improved dependency handling to reduce the risk of runtime errors such as class incompatibilities. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent b499a76 commit 656d3ed

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

tools/build_rules/flink/BUILD

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,6 @@ java_binary(
66
name = "flink",
77
main_class = "None", #hack
88
runtime_deps = [
9-
# mark the flink-streaming scala as provided as otherwise we end up with some extra Flink classes in our jar
10-
# and errors at runtime like: java.io.InvalidClassException: org.apache.flink.streaming.api.scala.DataStream$$anon$1; local class incompatible
11-
scala_artifact("org.apache.flink:flink-streaming-scala"),
129
# Exclude Guava from the assembled JAR
1310
# Else we hit an error - java.lang.NoSuchMethodError: com.google.common.base.Preconditions.checkArgument(...)
1411
maven_artifact("com.google.guava:guava"),

0 commit comments

Comments
 (0)