dongjoon-hyun commented on a change in pull request #23463:
[SPARK-26536][BUILD][FOLLOWUP][test-maven] Make StreamingReadSupport public for
maven testing
URL: https://github.com/apache/spark/pull/23463#discussion_r245535101
##########
File path:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/streaming/StreamingReadSupport.java
##########
@@ -20,11 +20,10 @@
import org.apache.spark.sql.sources.v2.reader.ReadSupport;
/**
- * A base interface for streaming read support. This is package private and is
invisible to data
- * sources. Data sources should implement concrete streaming read support
interfaces:
- * {@link MicroBatchReadSupport} or {@link ContinuousReadSupport}.
+ * A base interface for streaming read support. Data sources should implement
concrete streaming
+ * read support interfaces: {@link MicroBatchReadSupport} or {@link
ContinuousReadSupport}.
*/
-interface StreamingReadSupport extends ReadSupport {
+public interface StreamingReadSupport extends ReadSupport {
Review comment:
`VisibleForTesting` is added. And, yes. I agree that this is a weird and
totally unexpected situation. I've investigated why this happens in Maven
testing only and tried to find some Maven-dependent solutions. Until now, there
was no luck.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
