andygrove commented on code in PR #2169: URL: https://github.com/apache/datafusion-comet/pull/2169#discussion_r2282957342
########## spark/src/main/scala/org/apache/comet/CometExecIterator.scala: ########## @@ -35,22 +35,66 @@ import org.apache.comet.Tracing.withTrace import org.apache.comet.vector.NativeUtil /** - * An iterator class used to execute Comet native query. It takes an input iterator which comes - * from Comet Scan and is expected to produce batches of Arrow Arrays. During consuming this - * iterator, it will consume input iterator and pass Arrow Arrays to Comet native engine by - * addresses. Even after the end of input iterator, this iterator still possibly continues - * executing native query as there might be blocking operators such as Sort, Aggregate. The API - * `hasNext` can be used to check if it is the end of this iterator (i.e. the native query is - * done). + * Comet's primary execution iterator that bridges JVM (Spark) and native (Rust) execution Review Comment: Thanks. i've pushed a more concise version -- 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
