realno commented on pull request #1560: URL: https://github.com/apache/arrow-datafusion/pull/1560#issuecomment-1016114277
> LGTM over all. Left a comment on whether we should still keep the old polling logic around. > > Very well written design doc and nice benchmark numbers @yahoNanJing 👍 @houqp I have a slight concern that the push model adds some extra complexities to the design - please see my comment above. Perhaps we can have a discussion before merging this? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
