[GitHub] [spark] cloud-fan commented on issue #25180: [SPARK-28423][SQL] Merge Scan and Batch/Stream

2019-07-25 Thread GitBox
cloud-fan commented on issue #25180: [SPARK-28423][SQL] Merge Scan and Batch/Stream URL: https://github.com/apache/spark/pull/25180#issuecomment-515283197 I think the separation between `Scan` and `Batch` is still useless even if we move the operator pushdown to the optimizer. There is no

[GitHub] [spark] cloud-fan commented on issue #25180: [SPARK-28423][SQL] Merge Scan and Batch/Stream

2019-07-24 Thread GitBox
cloud-fan commented on issue #25180: [SPARK-28423][SQL] Merge Scan and Batch/Stream URL: https://github.com/apache/spark/pull/25180#issuecomment-514548375 retest this please This is an automated message from the Apache Git Se

[GitHub] [spark] cloud-fan commented on issue #25180: [SPARK-28423][SQL] Merge Scan and Batch/Stream

2019-07-23 Thread GitBox
cloud-fan commented on issue #25180: [SPARK-28423][SQL] Merge Scan and Batch/Stream URL: https://github.com/apache/spark/pull/25180#issuecomment-514434514 @rdblue cool, thanks! This is an automated message from the Apache Git

[GitHub] [spark] cloud-fan commented on issue #25180: [SPARK-28423][SQL] merge Scan and Batch/Stream

2019-07-17 Thread GitBox
cloud-fan commented on issue #25180: [SPARK-28423][SQL] merge Scan and Batch/Stream URL: https://github.com/apache/spark/pull/25180#issuecomment-512446883 @jose-torres seems there is a misunderstanding. This is on my TODO list a long time ago and I do want to finish it before 3.0. ---

[GitHub] [spark] cloud-fan commented on issue #25180: [SPARK-28423][SQL] merge Scan and Batch/Stream

2019-07-17 Thread GitBox
cloud-fan commented on issue #25180: [SPARK-28423][SQL] merge Scan and Batch/Stream URL: https://github.com/apache/spark/pull/25180#issuecomment-512330360 cc @marmbrus @rdblue @jose-torres @gengliangwang This is an automated