TheNeuralBit commented on pull request #14786: URL: https://github.com/apache/beam/pull/14786#issuecomment-843480570
> can you please add a unit test to verify the groupby bug you are fixing? Thanks, good idea. Testing the apply bug directly uncovered another edge case that wasn't being handled. Pushed two commits: - bc898ec adds new test case for column order bug - 332045b adds new test case and fix for an issue with determining when to include grouped columns in the output index I also rebased to resolve merge conflicts. @rohdesamuel would you mind taking another look at the two new commits since it includes a functionality change? -- 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. For queries about this service, please contact Infrastructure at: [email protected]
