steveloughran edited a comment on issue #1899: URL: https://github.com/apache/hadoop/pull/1899#issuecomment-618374352
> Sorry for the force push, I had done the -amend and rebased it afterwards. So, wasn't able to go back to the previous HEAD to have the commit and unstage changes. why the -amend? Why not just add another cvhange. Please don't rebase once reviewing has started, as it becomes impossible to tie discussions back to the state of the patch at the time, or see what changes happened after. For example, a commit called "fix review changes" Which review? I can't see from the history any more Until other people start reviewing -go for it. Once it's begun, if you do need to reset everything it is better to start again with a whole new PR with the single history squashed ``` git diff trunk...HEAD > history.diff git co trunk git co -b new-branch git apply -3 --verbose --whitespace=fix history.diff ..etc. etc ``` Key point; rebasing makes reviewing significantly harder, the harder a patch is to review, the fewer reviews it gets. ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
