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

2013-07-29 Thread Federico Cavalieri
I did not find a description of the process necessary to propose a new module in the wiki. What should I do? Thanks -- https://code.launchpad.net/~zorba-coders/zorba/feature-cloudant/+merge/177111 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~z

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

2013-07-29 Thread Matthias Brantner
> Is there an easier way to do what's currently shown on lines 1309-1317 of the > diff, i.e., ask "Is this atomic item *any* form of string?" See src/runtime/numerics/numerics_impl.cpp:73 > > Is there an easier way to do what's currently shown on lines 1407-1433 of the > diff, i.e., ask, "Is this

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

2013-07-29 Thread Matthias Brantner
Review: Approve I approve the changes now. I think that this still needs a little work if we start pushing-down the dataguide into collections. -- https://code.launchpad.net/~zorba-coders/zorba/use-dataguide/+merge/176385 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing lis

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

2013-07-29 Thread Paul J. Lucas
> What is not compatible with JSON? So it's OK to have, say, a base64Binary value as-is in an array or as a value in a key/value pair? -- https://code.launchpad.net/~zorba-coders/zorba/bug-1200090/+merge/177233 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://l

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module

2013-07-29 Thread Luis Rodriguez Gonzalez
Luis Rodriguez Gonzalez has proposed merging lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module. Requested reviews: Luis Rodriguez Gonzalez (kuraru) Chris Hillery (ceejatec) Related bugs: Bug #1189784 in Zorba: "Update non-core module "sqlite"" https://bugs.launchpad.net/zorba

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Chris Hillery
Review: Approve Global Items code looks good now! As a general rule, don't put actual function definitions into header files. Certain small methods (accessors, usually) can be inline for efficiency, but anything more significant can cause code bloat and other problems. I moved the ArchiveModul

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Zorba Build Bot
Voting criteria failed for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 : Votes: {'Approve': 1, 'Pending': 1, 'Needs commit message': 1} -- https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Your team Zorba Coders is

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Your team Zorba Cod

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Zorba Build Bot
Validation queue result for https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Stage "CommitZorba" failed. Check console output at http://jenkins.lambda.nu/job/CommitZorba/66/console to view the results. -- https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module

2013-07-29 Thread Chris Hillery
Review: Approve Looks good - nice job! I made three small changes: 1. Moved a in the metadata() doc to immediately after the leading sentence of the comment. 2. Moved SqliteModule() constructor definition out-of-line. 3. Per mailing list discussion, marked a number of module functions as no

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module

2013-07-29 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 Your team Zorba Cod

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module

2013-07-29 Thread Zorba Build Bot
Voting criteria failed for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 : Votes: {'Approve': 1, 'Pending': 1, 'Needs commit message': 1} -- https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 Your team Zorba Coders is

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module

2013-07-29 Thread Zorba Build Bot
Validation queue result for https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 Stage "CommitZorba" failed. Check console output at http://jenkins.lambda.nu/job/CommitZorba/67/console to view the results. -- https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge

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

2013-07-29 Thread Chris Hillery
There isn't really a fixed process for that since it happens so rarely. Here's what I'd suggest: 1. Create the module as a stand-alone directory, so that it works when located in the zorba_modules directory as part of a Zorba build. 2. Push that module to Launchpad as lp:~zorba-coders/zorba/clo

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Luis Rodriguez Gonzalez
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Your team Zorba Coders is subscribed to branch lp:zorba/archive-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpa

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module

2013-07-29 Thread Luis Rodriguez Gonzalez
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 Your team Zorba Coders is subscribed to branch lp:zorba/sqlite-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Your team Zorba Cod

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Zorba Build Bot
Voting criteria failed for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 : Votes: {'Approve': 2, 'Needs commit message': 1} -- https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Your team Zorba Coders is subscribed to

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Luis Rodriguez Gonzalez
The proposal to merge lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module has been updated. Commit Message changed to: JSONiqfication for Archive Module Done. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 -- https://code.launchpad.n

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Zorba Build Bot
Validation queue result for https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Stage "CommitZorba" failed. Check console output at http://jenkins.lambda.nu/job/CommitZorba/68/console to view the results. -- https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module

2013-07-29 Thread Luis Rodriguez Gonzalez
The proposal to merge lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module has been updated. Commit Message changed to: JSONiqfication for Sqlite Module Done! For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 -- https://code.launchpad.net

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Your team Zorba Cod

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread Zorba Build Bot
Validation queue succeeded - proposal merged! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 Your team Zorba Coders is subscribed to branch lp:zorba/archive-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Un

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module

2013-07-29 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug-1188033 into lp:zorba/archive-module has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1188033/+merge/176523 -- https://code.launchpad.net/~zorba-coders/zorba/bug-118803

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module

2013-07-29 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 Your team Zorba Cod

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module

2013-07-29 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1189784

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1189784 into lp:zorba/sqlite-module

2013-07-29 Thread Zorba Build Bot
Validation queue succeeded - proposal merged! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1189784/+merge/177489 Your team Zorba Coders is subscribed to branch lp:zorba/sqlite-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Uns

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

2013-07-29 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/python_installer_fix/+merge/174895 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/python_installer_fix/+merge/174895 Y

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

2013-07-29 Thread Zorba Build Bot
Validation queue succeeded - proposal merged! -- https://code.launchpad.net/~zorba-coders/zorba/python_installer_fix/+merge/174895 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscr

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

2013-07-29 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/python_installer_fix into lp:zorba has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/python_installer_fix/+merge/174895 -- https://code.launchpad.net/~zorba-coders/zorba/python_