Re: [PR] [AURON #2491] Carry Spark's logical link onto converted native plans [auron]

["robreeves (via GitHub)" <gi...@apache.org>]

robreeves commented on code in PR #2492:
URL: https://github.com/apache/auron/pull/2492#discussion_r4294405251


##########
spark-extension-shims-spark/src/test/scala/org/apache/auron/AuronQuerySuite.scala:
##########
@@ -1144,4 +1147,88 @@ class AuronQuerySuite extends AuronQueryTest with BaseAuronSQLSuite with AuronSQ
       checkSparkAnswerAndOperator("select * from orc_or where id = 5 or b = 900000")
     }
   }
+

Review Comment:
   Is there any interaction with `Dataset.cache`? This will have a logical plan, but hides the original plan from downstream operators. If relevant, could you add a test for it?



-- 
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: commits-unsubscribe@auron.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org