mosche commented on code in PR #22584:
URL: https://github.com/apache/beam/pull/22584#discussion_r982579185
##########
sdks/java/io/cdap/build.gradle:
##########
@@ -52,14 +52,17 @@ dependencies {
implementation library.java.cdap_plugin_zendesk
implementation library.java.commons_lang3
implementation library.java.guava
+ implementation library.java.google_code_gson
implementation library.java.hadoop_common
implementation library.java.hadoop_mapreduce_client_core
implementation library.java.jackson_core
implementation library.java.jackson_databind
implementation library.java.slf4j_api
+ implementation library.java.spark_streaming
Review Comment:
@Amar3tto Unfortunately this can't be dynamically resolved. If we
(eventually) want to support Spark 2 and 3, there has to be 2 maven artifacts.
I was talking about a naming scheme for these.
So something like:
- org.apache.beam/beam-sdks-java-io-cdap-spark2
- org.apache.beam/beam-sdks-java-io-cdap-spark3
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
