Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug938320 into lp:zorba

2012-08-28 Thread Chris Hillery
Ah, ok, I get it. Thanks for the explanation. I'll review the rest of the code soon. -- https://code.launchpad.net/~zorba-coders/zorba/bug938320/+merge/118194 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug938320 into lp:zorba

2012-08-28 Thread Cezar Andrei
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug938320/+merge/118194 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 : https://launchpad.net/~zorba-code

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug938320 into lp:zorba

2012-08-28 Thread Rodolfo Ochoa
@Chris ZorbaIOStream is the base class that should be used in other languages to stream to/from Zorba, it works as a wrapper for other language stream classes, for example, in Java this class becomes: ZorbaInputWrapper to manage InputStream Objects ZorbaOutputWrapper to manage OutputStream Object

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba

2012-08-28 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-execute-scripts/+merge/121706 -- https://code.launchpad.net/~zorba-coders/zorba/fix-execute-s

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba

2012-08-28 Thread Zorba Build Bot
Validation queue job fix-execute-scripts-2012-08-28T21-01-12.403Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/fix-execute-scripts/+merge/121706 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launch

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba

2012-08-28 Thread Juan Zacarias
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix-execute-scripts/+merge/121706 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 : https://launchpad.net/~

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba

2012-08-28 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix-execute-scripts/+merge/121706 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 : https://launchpad.net/~

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba

2012-08-28 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/fix-execute-scripts-2012-08-28T21-01-12.403Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/fix-execute-scripts/+merge/121706 Your team Zorba Coders is subscribed to branch lp:zorba. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bang_operator into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merg

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 2 Approve, 2 Pending. -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread Zorba Build Bot
Validation queue job bang_operator-2012-08-28T20-47-02.025Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zor

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba

2012-08-28 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-execute-scripts/+merge/121706 -- https://code.launchpad.net/~zorba-coders/zorba/fix-exe

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba

2012-08-28 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/fix-execute-scripts/+merge/121706 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 : https://launchpad.net/~

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba

2012-08-28 Thread Chris Hillery
Chris Hillery has proposed merging lp:~zorba-coders/zorba/fix-execute-scripts into lp:zorba. Requested reviews: Chris Hillery (ceejatec) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/fix-execute-scripts/+merge/121706 -- https://code.launchpad.net/~zorba-coders/zorba/fi

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bang_operator-2012-08-28T20-47-02.025Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing lis

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread William Candillon
The proposal to merge lp:~zorba-coders/zorba/bang_operator into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merg

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1042840 into lp:zorba

2012-08-28 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug-1042840 into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1042840/+merge/121682 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1042840/+merge/121682 You

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1042840 into lp:zorba

2012-08-28 Thread Zorba Build Bot
Validation queue job bug-1042840-2012-08-28T19-42-04.317Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1042840/+merge/121682 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-c

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1042840 into lp:zorba

2012-08-28 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1042840-2012-08-28T19-42-04.317Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1042840/+merge/121682 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: h

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1042840 into lp:zorba

2012-08-28 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/bug-1042840 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1042840/+merge/121682 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1042840/+merge/1216

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1042840 into lp:zorba

2012-08-28 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1042840/+merge/121682 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 : https://launchpad.net/~zorba-co

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1042840 into lp:zorba

2012-08-28 Thread Markos Zaharioudakis
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/bug-1042840 into lp:zorba. Requested reviews: Markos Zaharioudakis (markos-za) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1042840/+merge/121682 Fixed bug #1042840 (qname pool free-list corruption)

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/feature-uuid into lp:zorba

2012-08-28 Thread Paul J. Lucas
I added a mention to the build documentation. Approve now? -- https://code.launchpad.net/~zorba-coders/zorba/feature-uuid/+merge/118268 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Un

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-static-casts into lp:zorba

2012-08-28 Thread Markos Zaharioudakis
Ghislain, it seems that the jsoniq tree id is not currently used anywhere. Do you envision that it will ever be needed somewhere? If not, let's remove it. Also, I don't like the "fix" "unfix" names. What about addInCollection and removeFromCollection? (but this is up to you, I will approve anywa

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug938320 into lp:zorba

2012-08-28 Thread Cezar Andrei
Review: Needs Fixing Isn't the following method wrong? You need to check that the strings are the same not just a subset one of another. Just say: return (test == null && query.result == null) || test.equals(queryRTesult); 2068+ static boolean checkResult(String test, String queryResult) {

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bang_operator into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merg

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bang_operator into lp:zorba failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 (message): Validation queue job bang_operator-2012-08-28T14-03-07.533Z is finished. The fin

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bang_operator-2012-08-28T14-03-07.533Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing lis

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread William Candillon
The proposal to merge lp:~zorba-coders/zorba/bang_operator into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merg

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/jsoniq-static-casts into lp:zorba

2012-08-28 Thread Ghislain Fourny
Thanks Matthias, I missed this one! -- https://code.launchpad.net/~zorba-coders/zorba/jsoniq-static-casts/+merge/118961 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 : https

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bang_operator into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merg

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread Zorba Build Bot
Attempt to merge into lp:zorba failed due to conflicts: text conflict in ChangeLog -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-cod

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-08-28 Thread William Candillon
The proposal to merge lp:~zorba-coders/zorba/bang_operator into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merg

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1038410 into lp:zorba

2012-08-28 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug-1038410 into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1038410/+merge/121543 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1038410/+merge/121543 You

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1038410 into lp:zorba

2012-08-28 Thread Zorba Build Bot
Validation queue job bug-1038410-2012-08-28T07-23-03.04Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1038410/+merge/121543 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-co

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1038410 into lp:zorba

2012-08-28 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1038410-2012-08-28T07-23-03.04Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1038410/+merge/121543 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: ht

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1038410 into lp:zorba

2012-08-28 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/bug-1038410 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1038410/+merge/121543 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1038410/+merge/1215

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1038410 into lp:zorba

2012-08-28 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1038410/+merge/121543 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 : https://launchpad.net/~zorba-co

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1038410 into lp:zorba

2012-08-28 Thread Markos Zaharioudakis
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/bug-1038410 into lp:zorba. Requested reviews: Markos Zaharioudakis (markos-za) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1038410/+merge/121543 Fixed bug #1038410 (Memory leaks in parser, trace ite