Code does not compile. I would fix if this branch was owned by ~zorba-coders,
but since it's ~danielturcanu, we'll need Daniel to fix it.
[ 13%] Building CXX object
src/CMakeFiles/zorba_simplestore.dir/api/xqueryimpl.cpp.o
.In file included from ../../src/api/xqueryimpl.cpp:58:
In my debugging I saw that the flwor_expr is outside the udf. So when compiling
a udf, probably the optimizer is changing the expression tree in other places
as well, probably the caller of the udf, but I cannot say for sure. This
woulnd't surprise me, it should be normal for an optimizer.
--
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/89025
--
The attempt to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:273
(message):
Validation queue job plan-serializer-2012-01-18T16-04-09.348Z is finished.
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/89025
--
zorba_data-converters_module/converters/base64/binaryReadWrite.xq Failed
/home/ceej/zo/testing/zorbatest/tester/remotequeue/plan-serializer-2012-01-18T16-04-09.348Z/bzr/zorba/build/src/libzorba_simplestore.so.2.1.0
[0x7fde24a51399]
I would like to understand this better. What is the problem that this change
fixes?
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/88783
Your team Zorba Coders is requested to review the proposed merge of
lp:~danielturcanu/zorba/plan-serializer into lp:zorba.
--
The user_functions were compiled during plan serialization, and this modified
some expr trees that were already serialized. I had to ensure that all
user_functions are compiled before plan serialization is started.
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/88783
For example in one test, in flwor_expr object, theClauses was receiving another
flwor_clause in the vector, which made the vector to increase and move to
another space. But the initial flwor_clause had already been serialized, and
its address was registered for address duplication detection.
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/88783
--
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Commit Message changed to:
Fixing plan serializer memory leaks
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86959
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/plan-serializer-2011-12-29T08-09-47.208Z/log.html
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86959
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Validation queue job plan-serializer-2011-12-29T08-09-47.208Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86959
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86959
--
Review: Approve
+one
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86959
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 :
FYI, Daniel/Matthias - this proposal will need a commit message in order to get
merged...
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86959
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
Review: Approve
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86959
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 :
Daniel Turcanu has proposed merging lp:~danielturcanu/zorba/plan-serializer
into lp:zorba.
Requested reviews:
Zorba Coders (zorba-coders)
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86807
Fixed memory leak in serialization of Diagnostic
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86807
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/plan-serializer-2011-12-23T14-18-07.233Z/log.html
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86807
Your team Zorba Coders is requested to review the proposed merge of
Validation queue job plan-serializer-2011-12-23T14-18-07.233Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86807
Your team Zorba Coders is requested to review the proposed merge of
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1.
Got: 1 Pending.
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86807
Your team Zorba Coders is requested to review the proposed merge of
lp:~danielturcanu/zorba/plan-serializer into
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86807
--
Daniel, could you please commit the fix for the plan serializer separately from
disabling the module test? Otherwise it will be hard for Chris to find out what
you did.
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86807
Your team Zorba Coders is requested to review
Daniel Turcanu has proposed merging lp:~danielturcanu/zorba/plan-serializer
into lp:zorba.
Requested reviews:
Zorba Coders (zorba-coders)
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
Fixed plan serialization of udf_function
--
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/plan-serializer-2011-12-22T14-45-09.912Z/log.html
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
Your team Zorba Coders is requested to review the proposed merge of
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1.
Got: 1 Pending.
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
Your team Zorba Coders is requested to review the proposed merge of
lp:~danielturcanu/zorba/plan-serializer into
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
--
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
--
Validation queue job plan-serializer-2011-12-22T17-05-15.429Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
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 Approve, 2 Pending.
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Review: Approve
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
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:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/plan-serializer-2011-12-23T06-19-09.247Z/log.html
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Validation queue job plan-serializer-2011-12-23T06-19-09.247Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/86714
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/plan-serializer-2011-11-21T23-20-41.803Z/log.html
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The attempt to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:272
(message):
Validation queue job plan-serializer-2011-11-21T23-20-41.803Z is finished.
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
--
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/plan-serializer-2011-11-22T03-33-01.027Z/log.html
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Validation queue job plan-serializer-2011-11-22T03-33-01.027Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
--
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
--
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Commit Message changed to:
Fixed bug #872850 about plan serializer crash
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
--
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in ChangeLog
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
--
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in ChangeLog
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
--
Review: Approve
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
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/~danielturcanu/zorba/plan-serializer/+merge/79162
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 :
Daniel Turcanu has proposed merging lp:~danielturcanu/zorba/plan-serializer
into lp:zorba.
Requested reviews:
Zorba Coders (zorba-coders)
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79528
Fixed bug #872850 about plan serializer crash
Updated
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
Daniel Turcanu has proposed merging lp:~danielturcanu/zorba/plan-serializer
into lp:zorba.
Requested reviews:
Zorba Coders (zorba-coders)
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79160
Fixed bug #872850 about plan serializer crash
--
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79160
--
Attempt to merge into lp:zorba failed due to conflicts:
text conflict in ChangeLog
--
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79160
Your team Zorba Coders is requested to review the proposed merge of
lp:~danielturcanu/zorba/plan-serializer into lp:zorba.
--
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79160
--
Daniel Turcanu has proposed merging lp:~danielturcanu/zorba/plan-serializer
into lp:zorba.
Requested reviews:
Zorba Coders (zorba-coders)
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79162
Fixed bug #872850 about plan serializer crash
--
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/79162
--
60 matches
Mail list logo