The proposal to merge lp:~fcavalieri/zorba/structuralrelationships2 into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/structuralrelationships2/+merge/78395
--
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in ChangeLog
--
https://code.launchpad.net/~fcavalieri/zorba/structuralrelationships2/+merge/78395
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
The proposal to merge lp:~fcavalieri/zorba/structuralrelationships2 into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/structuralrelationships2/+merge/78395
--
The proposal to merge lp:~fcavalieri/zorba/structuralrelationships2 into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/structuralrelationships2/+merge/78395
--
There are additional revisions which have not been approved in review. Please
seek review and approval of these new revisions.
--
https://code.launchpad.net/~fcavalieri/zorba/structuralrelationships2/+merge/78395
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~fcavalieri/zorba/structuralrelationships2 into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/structuralrelationships2/+merge/78395
--
The proposal to merge lp:~fcavalieri/zorba/structuralrelationships2 into
lp:zorba has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/structuralrelationships2/+merge/78395
--
There are additional revisions which have not been approved in review. Please
seek review and approval of these new revisions.
--
https://code.launchpad.net/~fcavalieri/zorba/structuralrelationships2/+merge/78395
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~fcavalieri/zorba/structuralrelationships2 into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/structuralrelationships2/+merge/78395
--
There are additional revisions which have not been approved in review. Please
seek review and approval of these new revisions.
--
https://code.launchpad.net/~fcavalieri/zorba/structuralrelationships2/+merge/78395
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~fcavalieri/zorba/updrevalidate into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
--
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/updrevalidate-2011-10-13T07-32-54.096Z/log.html
--
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job updrevalidate-2011-10-13T07-32-54.096Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1.
Got: 1 Pending.
--
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
The proposal to merge lp:~fcavalieri/zorba/updrevalidate into lp:zorba has been
updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
--
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/updrevalidate-2011-10-13T08-18-04.321Z/log.html
--
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1.
Got: 1 Pending.
--
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
The proposal to merge lp:~fcavalieri/zorba/updrevalidate into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
--
https://code.launchpad.net/~fcavalieri/zorba/updrevalidate/+merge/79187
Hello Daniel
Thanks for fixing the issue that fast. Unfortunately, I didn't approve the
patch because of two comments I have:
1) Does the fix in src/compiler/expression/expr.cpp belongs do the tumbling
window for fix or is this something else? If yes, we need an extra test for
this. Or even
function name too long? node-structural-information
Any suggestions?
node-information
node-position
node-uri
structural-information
I like node-information as name for both - the module and the function.
That's at least consistent with the node-reference module and
In XQTS_1_0_3 the expected result for this test is 'true'.
According to the:
- W3C FO Spec:
http://www.w3.org/TR/xpath-functions-30/#func-namespace-uri-for-prefix
- decisions taken in bug 11590:
http://www.w3.org/Bugs/Public/show_bug.cgi?id=11590
The correct result in this case is the empty
Sorin Marian Nasoi has proposed merging lp:~zorba-coders/zorba/bug_872732 into
lp:zorba.
Requested reviews:
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug_872732/+merge/79272
Fixed the bug #872732 according to the:
- W3C spec:
function name too long? node-structural-information
Any suggestions?
node-information
node-position
node-uri
structural-information
I like node-information as name for both - the module and the function.
That's at least consistent with the node-reference
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79187
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79187
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
The proposal to merge lp:~zorba-coders/zorba/updrevalidate into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79187
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/updrevalidate-2011-10-13T16-27-57.947Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79187
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
The proposal to merge lp:~zorba-coders/zorba/updrevalidate into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79187
--
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79187
** Changed in: zorba
Status: New = Fix Released
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/872799
Title:
validate-in-place can set incorrect types
Status in Zorba - The XQuery
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/updrevalidate
into lp:zorba.
Requested reviews:
Federico Cavalieri (fcavalieri)
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79331
--
Federico, I forgot to submit some cosmetic changes to this branch before
pushing the approve button. So, now I have created this new merge proposal.
Can you please review? Thanks!.
--
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79331
Your team Zorba Coders is subscribed
Sure, but to me it looks like no change has been made at all (Preview Diff
contains only the word Empty) and no In a few minutes the diff will be
updated. Maybe I just have to wait...
Yes I just had to wait
--
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79331
Your team
The proposal to merge lp:~markos-za/zorba/indexes into lp:zorba has been
updated.
Commit Message changed to:
1. Big reorg of the code for general indexes:
1.1 Unified code for creating hash- and tree-based index
1.2 Unified code among different kinds of probe functions
1.3 Improved performance
Why variable numUpdates in PULImpl::mergeUpdateList is declared, assigned but
never read?
Good question :) The easy answer is to just remove it. But I think it can
actually be used to replace the myList.size() in the 2 for loops at lines 1289
and 1308. Even though new entries can be added in
Why variable numUpdates in PULImpl::mergeUpdateList is declared, assigned
but
never read?
Good question :) The easy answer is to just remove it. But I think it can
actually be used to replace the myList.size() in the 2 for loops at lines 1289
and 1308. Even though new entries can be
Stupid question regarding the commit: why are there so many whitespace changes
in the diff?
--
https://code.launchpad.net/~zorba-coders/zorba/bug_872732/+merge/79272
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
36 matches
Mail list logo