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

2012-12-06 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug-1073978 into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1073978/+merge/138355 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1073978/+merge/1383

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

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

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

2012-12-06 Thread Zorba Build Bot
Validation queue job bug-1073978-2012-12-07T05-35-48.353Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1073978/+merge/138355 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-1073978 into lp:zorba

2012-12-06 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1073978-2012-12-07T05-35-48.353Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1073978/+merge/138355 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: h

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

2012-12-06 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/bug-1073978 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1073978/+merge/138355 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1073978/+merge/1383

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

2012-12-06 Thread Matthias Brantner
> Another small comment: wouldn't be more user friendly for these two functions > to deal with xs:string instead of xs:anyURI? I changed it. -- https://code.launchpad.net/~zorba-coders/zorba/feature-reference_module/+merge/138345 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailin

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

2012-12-06 Thread Matthias Brantner
> Code looks good. > Small question: Why non-persistent structured-item() have no node reference > and non-persistent node have one? memory management is harder for json items (especially in 28msec). JSONObjects don't have a tree if they are not in a collection. We need the tree to figure out if

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

2012-12-06 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/markos-scratch into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/138556 -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/1

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

2012-12-06 Thread Zorba Build Bot
Validation queue job markos-scratch-2012-12-06T22-59-53.623Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/138556 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~z

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/xml-in-json-indices into lp:zorba

2012-12-06 Thread Markos Zaharioudakis
Why is the isStructuredItem() method a virtual one? I don't think it needs to be. -- https://code.launchpad.net/~zorba-coders/zorba/xml-in-json-indices/+merge/122548 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorb

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

2012-12-06 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2012-12-06T22-59-53.623Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/138556 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing l

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

2012-12-06 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/markos-scratch into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/138556 -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+m

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

2012-12-06 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/138556 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

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

2012-12-06 Thread Markos Zaharioudakis
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos-scratch into lp:zorba. Commit message: 1. Better computation of the static type for global variables 2. Optimization of comparison operators when untypedAtomic items are involved. Requested reviews: Markos Zaharioudakis

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

2012-12-06 Thread Paul J. Lucas
I modified test flwor10.xq to be: declare namespace ext = "http://www.zorba-xquery.com/extensions";; variable $x := 0; (# ext:materialize #) { for $i in (1, 2, 3) let $y := { $x := $x + $i; () } return $x } ran it and got: Zorba error [zerr:ZXQP0002]: "false": assertion failed with the m

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

2012-12-06 Thread William Candillon
Another small comment: wouldn't be more user friendly for these two functions to deal with xs:string instead of xs:anyURI? -- https://code.launchpad.net/~zorba-coders/zorba/feature-reference_module/+merge/138345 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://l

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/feature-json_strip_array into lp:zorba

2012-12-06 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/feature-json_strip_array into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~paul-lucas/zorba/feature-json_strip_array/+merge/136826 -- https://code.launchpad.net/~paul-lucas/zorba/feature-j

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/feature-json_strip_array into lp:zorba

2012-12-06 Thread Zorba Build Bot
Validation queue job feature-json_strip_array-2012-12-06T14-45-02.357Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~paul-lucas/zorba/feature-json_strip_array/+merge/136826 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https:

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/feature-json_strip_array into lp:zorba

2012-12-06 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/feature-json_strip_array-2012-12-06T14-45-02.357Z/log.html -- https://code.launchpad.net/~paul-lucas/zorba/feature-json_strip_array/+merge/136826 Your team Zorba Coders is subscribed to branch lp:zor

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/feature-json_strip_array into lp:zorba

2012-12-06 Thread Paul J. Lucas
The proposal to merge lp:~paul-lucas/zorba/feature-json_strip_array into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~paul-lucas/zorba/feature-json_strip_array/+merge/136826 -- https://code.launchpad.net/~paul-lucas/zorba/fea

Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/feature-json_strip_array into lp:zorba

2012-12-06 Thread Ghislain Fourny
Review: Approve -- https://code.launchpad.net/~paul-lucas/zorba/feature-json_strip_array/+merge/136826 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.ne

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

2012-12-06 Thread William Candillon
Code looks good. Small question: Why non-persistent structured-item() have no node reference and non-persistent node have one? -- https://code.launchpad.net/~zorba-coders/zorba/feature-reference_module/+merge/138345 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https