alamb commented on issue #16904: URL: https://github.com/apache/datafusion/issues/16904#issuecomment-3200376511
> I've marked the docs PR as ready for review: [#17213](https://github.com/apache/datafusion/pull/17213) > > And I'll separately start work on the [proposed new API.](https://github.com/apache/datafusion/issues/16904#issuecomment-3193805380) FYI about the API, I suggest you do it incrementally: 1. Figure out how to add a new API that *DOES NOT* require changing all existing call sites (this will also be the upgrade path for existing code) 2. Update one operator (e.g. GroupByHashStream) to use the new API 3. Add an example (ideally in doc comments) that shows how to use the new API to produce the report @rluvaton desires Using the example, we can then make sure the API looks good / does what we need, and then we can figure out how to roll it out to others -- 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
