Re: [PR] chore: Re-organize shuffle writer code [datafusion-comet]

2025-02-25 Thread via GitHub
andygrove commented on PR #1439: URL: https://github.com/apache/datafusion-comet/pull/1439#issuecomment-2683575116 Thanks for the reviews @mbutrovich @kazuyukitanimura -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] chore: Re-organize shuffle writer code [datafusion-comet]

2025-02-25 Thread via GitHub
andygrove merged PR #1439: URL: https://github.com/apache/datafusion-comet/pull/1439 -- 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...@

Re: [PR] chore: Re-organize shuffle writer code [datafusion-comet]

2025-02-25 Thread via GitHub
andygrove commented on PR #1439: URL: https://github.com/apache/datafusion-comet/pull/1439#issuecomment-2682080872 @comphead @kazuyukitanimura could I get a committer review? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

Re: [PR] chore: Re-organize shuffle writer code [datafusion-comet]

2025-02-24 Thread via GitHub
mbutrovich commented on PR #1439: URL: https://github.com/apache/datafusion-comet/pull/1439#issuecomment-2679374261 This is a great help as we look to improve the shuffle performance. Thanks @andygrove! -- This is an automated message from the Apache Git Service. To respond to the messag

Re: [PR] chore: Re-organize shuffle writer code [datafusion-comet]

2025-02-22 Thread via GitHub
codecov-commenter commented on PR #1439: URL: https://github.com/apache/datafusion-comet/pull/1439#issuecomment-2676356680 ## [Codecov](https://app.codecov.io/gh/apache/datafusion-comet/pull/1439?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_ca

[PR] chore: Re-organize shuffle writer code [datafusion-comet]

2025-02-22 Thread via GitHub
andygrove opened a new pull request, #1439: URL: https://github.com/apache/datafusion-comet/pull/1439 ## Which issue does this PR close? N/A ## Rationale for this change In preparation for improving the shuffle writer, this PR simply moves some code out o