Validation queue starting for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/json-http-module/+merge/169579
Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue
--
https://code.launchpad.net/~zorba-coders/zorba/json-http-module/+merge/169579
Your
Voting criteria failed for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/json-http-module/+merge/169579 :
Votes: {'Approve': 2, 'Needs commit message': 1}
--
https://code.launchpad.net/~zorba-coders/zorba/json-http-module/+merge/169579
Your team Zorba Coders is
The proposal to merge lp:~zorba-coders/zorba/json-http-module into lp:zorba has
been updated.
Commit Message changed to:
Added JSON HTTP client module.
Fixed initialization, error reporting and encoding bugs in the XML HTTP client.
Reimplemented XML HTTP client on top of the JSON HTTP client
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/json-http-module/+merge/169579
Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue
--
https://code.launchpad.net/~zorba-coders/zorba/json-http-module/+merge/169579
Your
Validation queue result for
https://code.launchpad.net/~zorba-coders/zorba/json-http-module/+merge/169579
Stage CommitZorba failed.
Check console output at http://jenkins.lambda.nu/job/CommitZorba/54/console to
view the results.
--
Validation queue succeeded - proposal merged!
--
https://code.launchpad.net/~zorba-coders/zorba/json-http-module/+merge/169579
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: Needs Information
1. Do we need to promote this to version 2.0 of the EXpath module? Isn't the
public API exactly the same, as defined by EXpath? I understand the
implementation is different, but I don't believe that's relevant for module
versioning.
2. (Possibly the same question)
The proposal to merge lp:~zorba-coders/zorba/json-http-module into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/json-http-module/+merge/169579
--
Is the code replication exact, or very nearly exact? If so I think I'd go with
option 2. (If it was only 50-100 lines of code I'd pick option 1, but 600 lines
seems like rather a lot.) I suspect that the XML http-client will be with us
for quite some time.
I agree that option 3, while clean,
I have tried some basic queries and there is a huge performance improvement.
This is great.
However, I detected a memory leak and have one question.
- memory leak in translator.cpp:4823
I've fixed it.
- In the following query, no dataguide seems to be pushed into the parser.
Why?
The
- In ZorbaCollectionIterator::initCollection lines 307 and 326, you eventually
add skipCount twice. Something smells spooky here.
Yes, it seems it was added twice. I've fixed it.
I've also added the skip() function to the EvalIterator.
--
--
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/skip-items/+merge/174723
Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue
--
https://code.launchpad.net/~zorba-coders/zorba/skip-items/+merge/174723
Your team Zorba
Validation queue result for
https://code.launchpad.net/~zorba-coders/zorba/skip-items/+merge/174723
Stage TestZorbaUbuntu failed.
1 tests failed (8415 total tests run).
Check test results at
http://jenkins.lambda.nu/job/TestZorbaUbuntu/123/testReport/ to view the
results.
--
The proposal to merge lp:~zorba-coders/zorba/http-client-wrapper into lp:zorba
has been updated.
Status: Needs review = Work in progress
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/http-client-wrapper/+merge/176980
--
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/skip-items/+merge/174723
Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue
--
https://code.launchpad.net/~zorba-coders/zorba/skip-items/+merge/174723
Your team Zorba
Depends on lp:~zorba-coders/zorba/http-client-wrapper
--
https://code.launchpad.net/~zorba-coders/zorba/http-client-based-on-json-http-client/+merge/169578
Your team Zorba Coders is subscribed to branch lp:zorba/http-client-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to
To avoid replicating conversion code (600-700 lines) between the core XML http
client and the Expath http client as discussed in:
https://code.launchpad.net/~zorba-coders/zorba/http-client-based-on-json-http-client/+merge/169578
I introduced a wrapper module for doing all conversions.
--
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/http-client-wrapper/+merge/176980
Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue
--
https://code.launchpad.net/~zorba-coders/zorba/http-client-wrapper/+merge/176980
Validation queue result for
https://code.launchpad.net/~zorba-coders/zorba/http-client-wrapper/+merge/176980
Stage CommitZorba failed.
Check console output at http://jenkins.lambda.nu/job/CommitZorba/57/console to
view the results.
--
Voting criteria failed for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/skip-items/+merge/174723 :
Votes: {'Needs Information': 1, 'Approve': 1}
--
https://code.launchpad.net/~zorba-coders/zorba/skip-items/+merge/174723
Your team Zorba Coders is subscribed to
Voting criteria failed for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/http-client-wrapper/+merge/176980
:
Votes: {'Pending': 2, 'Needs commit message': 1}
--
https://code.launchpad.net/~zorba-coders/zorba/http-client-wrapper/+merge/176980
Your team Zorba
Validation queue result for
https://code.launchpad.net/~zorba-coders/zorba/skip-items/+merge/174723
Stage CommitZorba failed.
Check console output at http://jenkins.lambda.nu/job/CommitZorba/56/console to
view the results.
--
Federico Cavalieri has proposed merging
lp:~zorba-coders/zorba/expath-http-on-json-http into
lp:zorba/http-client-module.
Requested reviews:
Cezar Andrei (cezar-andrei)
Chris Hillery (ceejatec)
For more details, see:
The proposal to merge lp:~zorba-coders/zorba/expath-http-on-json-http into
lp:zorba/http-client-module has been updated.
Status: Needs review = Work in progress
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/expath-http-on-json-http/+merge/176996
--
Federico Cavalieri has proposed merging
lp:~zorba-coders/zorba/expath-http-on-json-http into
lp:zorba/http-client-module.
Requested reviews:
Cezar Andrei (cezar-andrei)
Chris Hillery (ceejatec)
For more details, see:
Ready for review.
This is updates the expath http client to be based on the wrapper module
introduced in lp:~zorba-coders/zorba/http-client-wrapper
The comments of the previous branch (outdated links, wrapper conversion module)
have been addressed.
--
The proposal to merge lp:~zorba-coders/zorba/http-client-wrapper into lp:zorba
has been updated.
Commit Message changed to:
HTTP client request/response conversion in a separate module
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/http-client-wrapper/+merge/176980
--
Apparently the Expath HTTP Client queries 2 and 3 fail on the zorba demo website
(http://zorbawebsite2.my28msec.com/html/demo)
This does not seem to depend on the latest http client merge which apparently
is not there yet.
Is this a known problem?
--
Review: Needs Information
- In the following query, no dataguide seems to be pushed into the parser.
Why?
The count() function is marked with the %explores-json annotation, because
pruning objects that reach the function will modify the returned result. That
is why the dataguide is
Chris Hillery has proposed merging lp:~zorba-coders/zorba/jdbc-mariadb-driver
into lp:zorba/jdbc-module.
Commit message:
Additional expected result when using MariaDB JDBC driver.
Requested reviews:
Chris Hillery (ceejatec)
For more details, see:
The proposal to merge lp:~zorba-coders/zorba/jdbc-mariadb-driver into
lp:zorba/jdbc-module has been updated.
Commit Message changed to:
Additional expected result when using MariaDB JDBC driver.
For more details, see:
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/jdbc-mariadb-driver/+merge/177036
Your team Zorba Coders is subscribed to branch lp:zorba/jdbc-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
Validation queue starting for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/jdbc-mariadb-driver/+merge/177036
Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue
--
https://code.launchpad.net/~zorba-coders/zorba/jdbc-mariadb-driver/+merge/177036
Voting criteria failed for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/jdbc-mariadb-driver/+merge/177036
:
Votes: {'Approve': 1}
--
https://code.launchpad.net/~zorba-coders/zorba/jdbc-mariadb-driver/+merge/177036
Your team Zorba Coders is subscribed to branch
Validation queue result for
https://code.launchpad.net/~zorba-coders/zorba/jdbc-mariadb-driver/+merge/177036
Stage CommitZorba failed.
Check console output at http://jenkins.lambda.nu/job/CommitZorba/58/console to
view the results.
--
Review: Needs Fixing
The following query crashes:
xquery version 3.0;
import module namespace ddl =
http://www.zorba-xquery.com/modules/store/dynamic/collections/ddl;;
import module namespace dml =
http://www.zorba-xquery.com/modules/store/dynamic/collections/dml;;
import module namespace e =
Review: Needs Information
I have no idea what problem this MP is supposed to solve; nor do I know what a
data guide is.
--
https://code.launchpad.net/~zorba-coders/zorba/use-dataguide/+merge/176385
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Also, you have at least one /p typo in archive.xq.
Always remember to run
ctest -R moduleDocumentation
before committing, to be sure that your XQDoc is syntactically correct and
that you didn't forget to document any parameters, etc.
Done.
--
38 matches
Mail list logo