asolimando commented on PR #3757: URL: https://github.com/apache/calcite/pull/3757#issuecomment-2115825722
> > Few more minor changes and for me we are good to go. > > Please don't force-push until the review process is on-going as it makes incremental reviewing more difficult. > > In this case I force-pushed because there was a merge conflict against the head. Would it be preferred if I waited for the review to finish before dealing with these? Unless for cases in which this is breaking CI then it's indeed preferable for the review process to be over. -- 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]
