You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@spark.apache.org by "Marcelo Masiero Vanzin (Jira)" <ji...@apache.org> on 2019/11/07 01:10:00 UTC
[jira] [Resolved] (SPARK-29635) Deduplicate test suites between
Kafka micro-batch sink and Kafka continuous sink
[ https://issues.apache.org/jira/browse/SPARK-29635?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Marcelo Masiero Vanzin resolved SPARK-29635.
--------------------------------------------
Fix Version/s: 3.0.0
Resolution: Fixed
Issue resolved by pull request 26292
[https://github.com/apache/spark/pull/26292]
> Deduplicate test suites between Kafka micro-batch sink and Kafka continuous sink
> --------------------------------------------------------------------------------
>
> Key: SPARK-29635
> URL: https://issues.apache.org/jira/browse/SPARK-29635
> Project: Spark
> Issue Type: Improvement
> Components: Structured Streaming
> Affects Versions: 3.0.0
> Reporter: Jungtaek Lim
> Assignee: Jungtaek Lim
> Priority: Minor
> Fix For: 3.0.0
>
>
> There's a comment in KafkaContinuousSinkSuite which is most likely explaining TODO:
> https://github.com/apache/spark/blob/37690dea107623ebca1e47c64db59196ee388f2f/external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaContinuousSinkSuite.scala#L35-L39
> {noformat}
> /**
> * This is a temporary port of KafkaSinkSuite, since we do not yet have a V2 memory stream.
> * Once we have one, this will be changed to a specialization of KafkaSinkSuite and we won't have
> * to duplicate all the code.
> */
> {noformat}
> Given latest master branch has V2 memory stream now, it is a good time to deduplicate two suites into one, via having base class and let these suites override necessary things.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@spark.apache.org
For additional commands, e-mail: issues-help@spark.apache.org