Though I do see commits that should not be there. Did someone cherry pick commits from Impala-ASF to Impala on public gerrit? We may have to forcibly roll those back.
On Thu, Aug 18, 2016 at 6:07 PM, Jim Apple <[email protected]> wrote: > I don't see this using "git fetch gerrit && git log gerrit/cdh5-trunk". > > What is the output of "git remote -v" for you? > > On Thu, Aug 18, 2016 at 6:03 PM, Wang, Youwei A <[email protected]> > wrote: >> Greetings, Jim. Yes, the Change-Id I8777cf76f04d34a46f53d53005412e0f1d63b5b7 >> does appear in my branch. And it also appears in the cdh5-trunk. As a proof, >> the following message is copied from the output of "git log" for cdh5-trunk: >> >> commit 4bc2f0557bce48b04897c0123eba598812931785 >> Author: Tim Armstrong <[email protected]> >> Date: Mon May 23 14:09:39 2016 -0700 >> >> IMPALA-3611: track unused Disk IO buffer memory >> >> Track I/O buffers against separate MemTrackers. This gives us better >> visibility into memory consumption from the debug webpage and from >> MemTracker consumption dumps. The immediate motivation was in trying to >> determine whether idle memory consumption of an impalad was caused by a >> memory leak. >> >> We add two trackers: for buffers cached in DiskIoMgr's free list, >> and another for clients that don't provide a MemTracker (the only >> one is BufferedBlockMgr, which will be removed at some point). >> >> The previous code "tracked" the buffers against the process-wide >> tracker, but it was a no-op outside of ASAN builds since the >> process-wide tracker took its value from TCMalloc. >> >> The test code required fixing because it assumed that buffers were >> always credited against the DiskIoMgr's tracker. This only made sense >> when the DiskIoMgr's tracker is the root process-wide tracker. >> >> Fix backend test logging for disk-io-mgr-test. >> >> Testing: >> Ran exhaustive tests. >> >> Change-Id: I8777cf76f04d34a46f53d53005412e0f1d63b5b7 >> Reviewed-on: http://gerrit.cloudera.org:8080/3799 >> Reviewed-by: Dan Hecht <[email protected]> >> Tested-by: Internal Jenkins >> (cherry picked from commit 17bf14417e3438d772b19111431453bdd537742a) >> >> PS: And I have conducted some experiments last night: >> 1. Create an empty VMWare workstation and install Debian 8.5 (the known >> latest version); >> 2. Install git; No other unrelated packages are installed; >> 3. git clone https://github.com/cloudera/Impala.git >> 4. cd Impala >> 5. git checkout cdh5-trunk >> 6. Follow every step from this page: >> https://github.com/cloudera/Impala/wiki/Using-Gerrit-to-submit-and-review-patches >> 7. Apply my patch to cdh5-trunk and then commit it; >> 8. git push gerrit HEAD:refs/for/cdh5-trunk >> Result: I got the exact same error as we discussed. >> >> I have also try to substitute the VMWare workstation using a Debian docker >> image in the step 1. >> And the remained steps are identical. The same error still happens. >> >> -----Original Message----- >> From: Jim Apple [mailto:[email protected]] >> Sent: Thursday, August 18, 2016 8:28 PM >> To: Wang, Youwei A <[email protected]> >> Cc: [email protected] >> Subject: Re: Failed to submit/update my patch >> >> Does Change-Id: I8777cf76f04d34a46f53d53005412e0f1d63b5b7 show up in your >> git history when trying to push? If it does, something is wrong, since this >> change should not have been merged into gerrit/cdh5-trunk. >> >> On Thu, Aug 18, 2016 at 12:16 AM, Wang, Youwei A <[email protected]> >> wrote: >>> Greetings, everyone. >>> Thank you for providing such great solution, @Todd and @Jim. >>> But I am afraid I have to say the 'rebase -i' approach doesn't work for me >>> and yes, I have made 100% sure that my patch is the only patch that show up >>> in my interactive rebase is my own. >>> >>> I have also tried another approach: I have diffed my branch and the >>> cdh5-trunk, then saved it to another place. After that, I switched to the >>> cdh5-trunk, run git pull, applied my patch, and then finally commit my >>> changes. Then I run "git push gerrit HEAD:refs/for/cdh5-trunk", still got >>> the same error. Since now I am working on the latest cdh5-trunk, I believe >>> the possibility that other branches introduce this error is eliminated. >>> However, this issue still happens. >>> >>> I am trying to use another clean machine to re-commit my change and check >>> whether that will happen then. >>> >>> -----Original Message----- >>> From: Todd Lipcon [mailto:[email protected]] >>> Sent: Thursday, August 18, 2016 12:48 PM >>> To: Jim Apple <[email protected]> >>> Cc: dev@impala <[email protected]> >>> Subject: Re: Failed to submit/update my patch >>> >>> Typically this happens if you've somehow done a rebase such that the >>> upstream patch is in your branch under a different git hash than was >>> actually committed. I'd recommend fetching from the upstream branch, doing >>> a 'rebase -i' against it, and making sure that the only patches that show >>> up in your interactive rebase are your own. >>> >>> -Todd >>> >>> On Wed, Aug 17, 2016 at 8:32 PM, Jim Apple <[email protected]> wrote: >>> >>>> +todd >>>> >>>> Very strange - I have never seen anything like that. >>>> >>>> Todd, have you seen anything like this before? >>>> >>>> On Wed, Aug 17, 2016 at 8:14 PM, Wang, Youwei A >>>> <[email protected]> >>>> wrote: >>>> > Greetings, everyone. >>>> > Just two days ago, I could submit/update my patch using following >>>> command successfully: >>>> > git push gerrit HEAD:refs/for/cdh5-trunk >>>> > >>>> > However, when I try this command to update my patch at this link today: >>>> > https://gerrit.cloudera.org/#/c/3081/ >>>> > >>>> > I have got an error message saying: >>>> > ! [remote rejected] HEAD -> refs/for/cdh5-trunk (change >>>> http://gerrit.cloudera.org:8080/3246 closed) >>>> > error: failed to push some refs to 'ssh://Hayabusa-intel@gerrit. >>>> cloudera.org:29418/Impala' >>>> > >>>> > It seems the push command tried to push my change to another closed >>>> change. >>>> > That is really weird. Since my local changeId is: >>>> > Git log -1 >>>> > Author: .................................... >>>> > Date: Thu Aug 18 22:53:13 2016 +0800 >>>> > >>>> > IMPALA-2809: Improve ByteSwap with builtin function or SSSE3 or AVX2. >>>> > .................................... >>>> > >>>> > Change-Id: I392ed5a8d5683f30f161282c228c1aedd7b648c1 >>>> > >>>> > The changed from my patch link >>>> > https://gerrit.cloudera.org/#/c/3081/ >>>> > is >>>> also: >>>> > Change-Id: I392ed5a8d5683f30f161282c228c1aedd7b648c1 >>>> > You can see they are identical. >>>> > >>>> > As for this link http://gerrit.cloudera.org:8080/3246, >>>> > Its changeId is I8777cf76f04d34a46f53d53005412e0f1d63b5b7. >>>> > Yes, it is closed so the error message (change >>>> http://gerrit.cloudera.org:8080/3246 closed) does make some sense here. >>>> But I have no idea why git pushes my change to another unrelated change. >>>> > >>>> > Thank you for any hint or idea. :) >>>> > >>>> > >>>> > >>>> >>> >>> >>> >>> -- >>> Todd Lipcon >>> Software Engineer, Cloudera
