parthchandra commented on PR #5365: URL: https://github.com/apache/datafusion-comet/pull/5365#issuecomment-5375342431
> Hey @sunchao / @parthchandra — are you looking to move this work over to @dwsmith1983’s series? > > We’re 2 PRs into the 10-PR series now that the contrib modules have been merged. The series fully implements all of the Delta protocols, with all 10k+ Delta test cases passing. > > I’m fine either way. I won’t have a huge amount of time to work on this over the next couple of months, so it could go faster with a different attendant, but the PRs are ready to roll. > > You can see the series here: https://github.com/schenksj/datafusion-comet/pulls (PRs #5–13). I see value in both (even though it is extra work to maintain both paths) and in principle agree with @sunchao. Ideally, we want to converge these two. Logged an issue based on an AI generated convergence path - https://github.com/apache/datafusion-comet/issues/5411 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
