rdblue commented on a change in pull request #24129: [SPARK-27190][SQL] add 
table capability for streaming
URL: https://github.com/apache/spark/pull/24129#discussion_r272395216
 
 

 ##########
 File path: sql/core/src/main/java/org/apache/spark/sql/sources/v2/Table.java
 ##########
 @@ -27,8 +27,8 @@
  * implementation can be a directory on the file system, a topic of Kafka, or 
a table in the
  * catalog, etc.
  * <p>
- * This interface can mixin the following interfaces to support different 
operations, like
- * {@code SupportsRead}.
+ * This interface can mixin {@link SupportsRead} and {@link SupportsWrite} to 
provide data reading
 
 Review comment:
   This is going to cause a conflict with #24246, which moves this class into 
catalyst. The links will no longer work after that move, so I'd rather avoid 
the conflict and make this change in that PR. That also make this PR touch one 
fewer file, so that's good. Could you remove this change?

----------------------------------------------------------------
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.
 
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]

Reply via email to