I think that the plan is still to release from the branch …

On 18 Nov 2015, at 17:27, Yingyi Bu wrote:

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

Reply via email to