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

2013-03-16 Thread Markos Zaharioudakis
Review: Needs Fixing The implicit iteration should always be done in the translator, not in the runtime (actually, that's how it is in the hof branch). So the new do-loop in dynamic_function_iterator.cpp should be removed. --

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

2013-03-16 Thread Markos Zaharioudakis
Review: Needs Fixing The implicit iteration should always be done in the translator, not in the runtime (actually, that's how it is in the hof branch). So the new do-loop in dynamic_function_iterator.cpp should be removed. --

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

2013-03-16 Thread Matthias Brantner
For both cases, the function invocation with 0 and 1 arguments? Does the dynamic function call itself (i.e. on function items) also do implicit iteration? If not, how does the translator decide whether implicit iteration has to be done or not? On Mar 16, 2013, at 11:32 AM, Markos Zaharioudakis

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

2013-03-16 Thread Markos Zaharioudakis
For both cases, the function invocation with 0 and 1 arguments? Does the dynamic function call itself (i.e. on function items) also do implicit iteration? If not, how does the translator decide whether implicit iteration has to be done or not? The current XQuery 3.0 spec does not talk

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

2013-03-16 Thread Markos Zaharioudakis
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos-scratch into lp:zorba. Commit message: Fixed bug in index join rule (no index join if inner clause has positional var). Requested reviews: Markos Zaharioudakis (markos-za) For more details, see:

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

2013-03-16 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/153669 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 :

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

2013-03-16 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/153669 --

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

2013-03-16 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2013-03-16T20-31-51.152Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/153669 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

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

2013-03-16 Thread Zorba Build Bot
Validation queue job markos-scratch-2013-03-16T20-31-51.152Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/153669 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2013-03-16 Thread Markos Zaharioudakis
Review: Approve looks ok to me, but i am not the most qualified person in this area ... -- https://code.launchpad.net/~zorba-coders/zorba/feature-jsoniq_parser/+merge/153648 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to

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

2013-03-16 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/153669 --

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

2013-03-16 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/feature-jsoniq_parser/+merge/153648 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 :

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

2013-03-16 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/feature-jsoniq_parser into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-jsoniq_parser/+merge/153648 --

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

2013-03-16 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/feature-jsoniq_parser-2013-03-16T21-03-57.062Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/feature-jsoniq_parser/+merge/153648 Your team Zorba Coders is subscribed to branch lp:zorba.

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

2013-03-16 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/feature-jsoniq_parser into lp:zorba failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message): Validation queue job feature-jsoniq_parser-2013-03-16T21-03-57.062Z is

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

2013-03-16 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/feature-jsoniq_parser into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/feature-jsoniq_parser/+merge/153648 --

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

2013-03-16 Thread Paul J. Lucas
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1153800/+merge/153675 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 :

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

2013-03-16 Thread Paul J. Lucas
The proposal to merge lp:~zorba-coders/zorba/bug-1153800 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1153800/+merge/153675 --

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

2013-03-16 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1153800-2013-03-17T05-02-51.05Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1153800/+merge/153675 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2013-03-16 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bug-1153800 into lp:zorba failed. Below is the output from the failed tests. CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message): Validation queue job bug-1153800-2013-03-17T05-02-51.05Z is finished. The final

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

2013-03-16 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bug-1153800 into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1153800/+merge/153675 --