asolimando commented on PR #4267: URL: https://github.com/apache/calcite/pull/4267#issuecomment-2765660981
> > @silundong unless strictly necessary (CI changes from `main` that can't be picked up otherwise, etc.) please avoid force-pushing while the review is in progress as it makes it difficult for reviewers to do incremental diff via GitHub UI. You can push as many commits as needed, you will be required to squash them before merging. > > @asolimando Sorry for making the review difficult. Should I revert the commit to the state of the last review and update the changes to the new commit? No worries, it's not a big deal, you can keep the existing as-is and just add more commits in the future, thanks for your understanding! -- 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]
