You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2023/01/18 03:51:27 UTC

[GitHub] [spark] maryannxue commented on pull request #39624: [SPARK-42101][SQL] Wrap InMemoryTableScanExec with QueryStage

maryannxue commented on PR #39624:
URL: https://github.com/apache/spark/pull/39624#issuecomment-1386446139

   Can you elaborate on what benefit we can get from making "InMemoryTableScanExec as a kind of QueryStage"? A query stage has a physical implication that it has data movement and materialization. I don't think it's a good idea to change its semantics for the purpose of UI fix.


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

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

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