[GitHub] [spark] xuanyuanking commented on pull request #29633: [SPARK-32782][SS] Refactor StreamingRelationV2 and move it to catalyst

[GitBox <gi...@apache.org>]

xuanyuanking commented on pull request #29633:
URL: https://github.com/apache/spark/pull/29633#issuecomment-686267468


   ```
   Does it couple with core module?
   ```
   @HeartSaVioR  Yes, the StreamingRelation couples with DataSource. You may find all the DataSource resolving work is in `FindDataSourceTable` so it's ok to put it in the core module. But for v2 relation, it's unbound with the specific data source, both `Table` and `TableProvider` is in the catalyst, so move it can benefit for our future extension on StreamingRelationV2.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org