comphead commented on code in PR #2375: URL: https://github.com/apache/datafusion-comet/pull/2375#discussion_r2349412810
########## spark/src/main/scala/org/apache/spark/sql/comet/CometExecUtils.scala: ########## @@ -48,13 +48,14 @@ object CometExecUtils { * partition. The limit operation is performed on the native side. */ def getNativeLimitRDD( + child: SparkPlan, Review Comment: this is confusing IMO. `child` is the plan and `childPlan` in fact is data -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org