>> Yingyi, this patch is based on release branch resealse-0.8.8, so any if you had the similar change in master it will be duplicated way until release branch is merged back to master. Ok, got it.
I remember Till and Ian recently discussed how to make the release-0.8.8. @Ian, @Till, what's the latest plan for release-0.8.8? It will still be based on the release branch or will be based on the master? Thanks! Best, Yingyi On Wed, Nov 18, 2015 at 5:15 PM, Ildar Absalyamov (Code Review) < [email protected]> wrote: > Ildar Absalyamov has posted comments on this change. > > Change subject: ASTERIXDB-1109: Fixed deletion of records from open > secondary index - Avoided creating calling record constructor in delete > pipeline - Fixed the case when multiple open indexes enforce the type of > the same field & covered it with tests > ...................................................................... > > > Patch Set 10: > > (1 comment) > > > https://asterix-gerrit.ics.uci.edu/#/c/461/10/asterix-algebra/src/main/java/org/apache/asterix/translator/CompiledStatements.java > File > asterix-algebra/src/main/java/org/apache/asterix/translator/CompiledStatements.java: > > Line 532: FLWOGRExpression flowgr = new > FLWOGRExpression(clauseList, var); > > Can you merge with the latest master and resolve the conflict? > Yingyi, this patch is based on release branch resealse-0.8.8, so any if > you had the similar change in master it will be duplicated way until > release branch is merged back to master. > > > -- > To view, visit https://asterix-gerrit.ics.uci.edu/461 > To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings > > Gerrit-MessageType: comment > Gerrit-Change-Id: I41bde91401f67918365de7df19dd2f0de20c73d2 > Gerrit-PatchSet: 10 > Gerrit-Project: asterixdb > Gerrit-Branch: release-0.8.8 > Gerrit-Owner: Ildar Absalyamov <[email protected]> > Gerrit-Reviewer: Ian Maxon <[email protected]> > Gerrit-Reviewer: Ildar Absalyamov <[email protected]> > Gerrit-Reviewer: Jenkins <[email protected]> > Gerrit-Reviewer: Taewoo Kim <[email protected]> > Gerrit-Reviewer: Yingyi Bu <[email protected]> > Gerrit-HasComments: Yes >
