Review: Needs Fixing
I tested it. Works really nice. I just have some little comments concerning the
code:
1)
Do we already have a bug entry for this:
336 // TODO: this was the initial implementation. This will have to
change
337 -this-sleep_(1000);
338 +
The attempt to merge lp:~zorba-coders/zorba/fix-declare-module-bug917981 into
lp:zorba failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:273
(message):
Validation queue job
The proposal to merge lp:~zorba-coders/zorba/fix-declare-module-bug917981 into
lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-declare-module-bug917981/+merge/88977
--
William Candillon has proposed merging lp:~zorba-coders/zorba/phpapi into
lp:zorba.
Requested reviews:
William Candillon (wcandillon)
Rodolfo Ochoa (rodolfo-ochoa)
Matthias Brantner (matthias-brantner)
For more details, see:
Code does not compile. I would fix if this branch was owned by ~zorba-coders,
but since it's ~danielturcanu, we'll need Daniel to fix it.
[ 13%] Building CXX object
src/CMakeFiles/zorba_simplestore.dir/api/xqueryimpl.cpp.o
.In file included from ../../src/api/xqueryimpl.cpp:58:
The proposal to merge lp:~zorba-coders/zorba/feature-json_parser into lp:zorba
has been updated.
Commit Message changed to:
New JSON parser and module.
Fixed the missing quote in the documentation.
For more details, see:
I have resubmitted the merge proposal after committing the fix for the JSON
tests.
--
https://code.launchpad.net/~zorba-coders/zorba/feature-json_parser/+merge/89008
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
Review: Needs Fixing
The documentation is missing some examples.
The example section at http://docs.basex.org/wiki/JSON_Module is a good place
to get inspired.
Why A chars have a backslash in error code descriptions:
ZJSE0001 if \a $xml is not a document or element node.
ZJSE0002 if \a $xml
Sorin Marian Nasoi has proposed merging
lp:~zorba-coders/zorba/feature_xqdoc_version_for_modules into lp:zorba.
Requested reviews:
Sorin Marian Nasoi (sorin.marian.nasoi)
William Candillon (wcandillon)
For more details, see:
The proposal to merge lp:~zorba-coders/zorba/feature_xqdoc_version_for_modules
into lp:zorba has been updated.
Commit Message changed to:
Added the information about the Zorba module version in the generated XQDoc
documentation.
For more details, see:
If there is a module that is missing it's isCore property, than this is an
error inside Zorba and we should open a separate bug for this.
--
https://code.launchpad.net/~zorba-coders/zorba/feature_xqdoc_version_for_modules/+merge/89013
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Public bug reported:
At a minimum, we wish to test with ZORBA_WITH_BIG_INTEGER set both on
and off.
More generally, we should support multiple build configurations, each
with potentially different sets of tests run. For example, we don't
necessarily need to run XQTS with every configuration.
**
We should probably replace this with the XQuery-based test driver that
is being developed.
** Changed in: zorba
Milestone: None = 3.0
** Changed in: zorba
Assignee: nobody (nobody-users) = Chris Hillery (ceejatec)
** Changed in: zorba
Status: New = Confirmed
--
You received
Public bug reported:
The OpenCSX module needs to be completed, specifically to fully support
encode/decode of XDM instances with validated data.
** Affects: zorba
Importance: High
Assignee: Chris Hillery (ceejatec)
Status: Confirmed
** Changed in: zorba
Milestone: None =
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/debugger_ctrl_c/+merge/89014
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 :
Public bug reported:
In order for this to work, XQFTTS must run clean - this means that all
failing tests should be marked as EXPECTED_FAILURE() for all supported
build flags.
I believe a lot of this work has been done, but it's not merged yet.
Also, there may be some outstanding issues with
Public bug reported:
There is no FOTS driver for Zorba yet.
FOTS stands for XQuery/XPath/XSLT 3.* Test Suite, please see summary at:
http://dev.w3.org/cvsweb/~checkout~/2011/QT3-test-
suite/Overview.html?rev=1.5;content-type=text%2Fhtml
For details about FOTS please see:
Probably the validation queue should NOT short-circuit - if one build
configuration fails to build, it should still try the rest. That way we
get the most information per queue run. Probably any egregious build
failures will cause all configurations to fail quickly, so there isn't
much time
Public bug reported:
Need JRE/JDK (at least) to get it to build, and then need to download
XQJ and run it.
** Affects: zorba
Importance: Medium
Assignee: Rodolfo Ochoa (rodolfo-ochoa)
Status: Confirmed
** Changed in: zorba
Milestone: None = 3.0
** Changed in: zorba
Rodolfo, I assume you have written a testdriver of some kind for testing
the Zorba XQJ interface - is that part of the Zorba build and ctest
configuration? If not, can you make it so?
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos-scratch
into lp:zorba.
Requested reviews:
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/89017
Fixed bug 917923 (bug in copying outer var
Validation queue job ignore-2012-01-18T11-53-53.868Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~ceejatec/zorba/ignore/+merge/86507
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/89017
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 :
The proposal to merge lp:~ceejatec/zorba/ignore into lp:zorba has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~ceejatec/zorba/ignore/+merge/86507
--
https://code.launchpad.net/~ceejatec/zorba/ignore/+merge/86507
Your team Zorba Coders is
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2012-01-18T12-25-59.032Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/89017
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Public bug reported:
To at least include punching new store::Item methods through the
zorba::Item API.
** Affects: zorba
Importance: High
Assignee: Chris Hillery (ceejatec)
Status: Confirmed
** Changed in: zorba
Assignee: (unassigned) = Chris Hillery (ceejatec)
**
Public bug reported:
For example, it would be cool if we could do something like this:
import module namespace f=http://www.zorba-
xquery.com/modules/resolvers;
declare function foo ()...
f:register-uri-mapper(foo#2)
..and then subsequent URI resolutions would invoke foo().
Open
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/89021
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 :
It will be necessary for each build+test configuration to submit results
as separate CDash builds, and the passing/failing tests for each build
should be linked from the validation queue's log.html.
--
You received this bug notification because you are a member of Zorba
Coders, which is the
Validation queue job markos-scratch-2012-01-18T12-25-59.032Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/89017
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
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/89017
--
** Changed in: zorba
Status: Triaged = In Progress
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/918157
Title:
Add XQFTTS to validation queue
Status in Zorba - The XQuery Processor:
Public bug reported:
Create a way to generate the conformance reports automatically in Zorba.
After this is done, the Validation queue can add a new check to see if
any regression was added with the proposed commit.
** Affects: zorba
Importance: Medium
Assignee: Sorin Marian Nasoi
Review: Approve
Locks good!
--
https://code.launchpad.net/~zorba-coders/zorba/debugger_ctrl_c/+merge/89014
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 :
In my debugging I saw that the flwor_expr is outside the udf. So when compiling
a udf, probably the optimizer is changing the expression tree in other places
as well, probably the caller of the udf, but I cannot say for sure. This
woulnd't surprise me, it should be normal for an optimizer.
--
** Changed in: zorba
Status: New = In Progress
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/918173
Title:
add auto generation of the conformance reports to lp:zorba
Status in Zorba -
Review: Needs Information
It looks very good.
Should the actual atomic type be present in the attribute element?
If you have by @id where @id can be 1 or 1, you might want to create two
index keys?
If a key returns (), you might want to distinguish with empty string for
probing later?
By
The proposal to merge lp:~zorba-coders/zorba/debugger_ctrl_c into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/debugger_ctrl_c/+merge/89014
--
The Elsevier copyright came from the xqdoc.xsd, which I think I used as a base
for the options file. I fixed it together with the other issues Matthias has
raised. I've also fixed a couple of other cases where malformed XMLs where
handled incorrectly. Resubmitting for merging.
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/debugger_ctrl_c-2012-01-18T14-07-01.556Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/debugger_ctrl_c/+merge/89014
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing
Public bug reported:
xquerydoc v0.1 was just released: we should make sure that Zorba at
least runs it successfully.
Also investigate is anything needs to be done in order to support it.
** Affects: zorba
Importance: Medium
Assignee: Sorin Marian Nasoi (sorin.marian.nasoi)
Public bug reported:
Run the w3c conformance generation for the xqueryx fulltext merge
proposal made by Daniel:
https://code.launchpad.net/~zorba-coders/zorba/xqueryx_fulltext
merge proposal is here:
https://code.launchpad.net/~zorba-coders/zorba/xqueryx_fulltext/+merge/88159
** Affects:
Validation queue job debugger_ctrl_c-2012-01-18T14-07-01.556Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/debugger_ctrl_c/+merge/89014
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~zorba-coders/zorba/debugger_ctrl_c into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/debugger_ctrl_c/+merge/89014
--
** Summary changed:
- Debugger: Interrupt execution with Ctrl-C and stop at next breakpoint
+ Debugger: Interrupt execution with Ctrl-C and stop at next breakable
expression
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
Matthias, the getNamespaceBindings method is rather inefficient as it will copy
the actual strings twice. And it should also pre-allocated the space for the
returned vector. I don't think this is going to be a major bottleneck, so I
have approved the proposal. But in the spirit of best
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug905035/+merge/88922
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 :
Doesn't seem to work for xqdoc
--
https://code.launchpad.net/~zorba-coders/zorba/feature-json_parser/+merge/89008
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 :
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/89025
--
The attempt to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:273
(message):
Validation queue job plan-serializer-2012-01-18T16-04-09.348Z is finished.
The proposal to merge lp:~danielturcanu/zorba/plan-serializer into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~danielturcanu/zorba/plan-serializer/+merge/89025
--
The proposal to merge lp:~zorba-coders/zorba/testsuites_update into lp:zorba
has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/89021
--
Sorin - could you update the commit message for the merge proposal? It seems
there are a number of changes here besides just marking XQFTTS expected
failures.
--
https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/89021
Your team Zorba Coders is subscribed to branch
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/testsuites_update-2012-01-18T16-39-59.934Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/89021
Your team Zorba Coders is subscribed to branch lp:zorba.
--
zorba_data-converters_module/converters/base64/binaryReadWrite.xq Failed
/home/ceej/zo/testing/zorbatest/tester/remotequeue/plan-serializer-2012-01-18T16-04-09.348Z/bzr/zorba/build/src/libzorba_simplestore.so.2.1.0
[0x7fde24a51399]
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/phpapi/+merge/89002
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
** Branch linked: lp:~zorba-coders/zorba/bug-867059
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/867059
Title:
Can not handle largest xs:unsignedLong values
Status in Zorba - The XQuery
The attempt to merge lp:~zorba-coders/zorba/testsuites_update into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:273
(message):
Validation queue job testsuites_update-2012-01-18T16-39-59.934Z is
Looks like this change depends on Daniel's XSLT change in order for XQueryX
Fulltext tests to work. Unfortunately I can't merge that branch
(lp:~zorba-coders/zorba/xqueryx_fulltext) until THIS branch is merged, and
vice-versa.
Sorin, I think the only way to break the circle is to merge the
The proposal to merge lp:~zorba-coders/zorba/testsuites_update into lp:zorba
has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/89021
--
Would xmldoc be a good prefix?
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/89038
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 :
It certainly is the closest allowed alternative...
--
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/89038
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 :
Matthias Brantner has proposed merging lp:~zorba-coders/zorba/bug905035 into
lp:zorba.
Requested reviews:
Hybridum (hybridum)
Markos Zaharioudakis (markos-za)
Rodolfo Ochoa (rodolfo-ochoa)
Related bugs:
Bug #905035 in Zorba: there is no way to get the Namespace Prefixes
The proposal to merge lp:~zorba-coders/zorba/bug905035 into lp:zorba has been
updated.
Commit Message changed to:
Implemented StaticContext::getNamespaceBindings to resolve bug #905035. In the
same commit, the function getNamespaceURIByPrefix was deprecated because it's
superseded by the new
Try it now.
--
https://code.launchpad.net/~zorba-coders/zorba/no_unicode/+merge/85142
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
The proposal to merge lp:~zorba-coders/zorba/phpapi into lp:zorba has been
updated.
Commit Message changed to:
This merge adds the PHP API that was introduced at the PHP Tour 2011.
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/phpapi/+merge/89002
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/phpapi-2012-01-18T18-48-01.633Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/phpapi/+merge/89002
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job phpapi-2012-01-18T18-48-01.633Z is finished. The final
status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/phpapi/+merge/89002
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post
The proposal to merge lp:~nbrinza/zorba/parse-fragment into
lp:~zorba-coders/zorba/zorba-experimental has been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/89108
--
The proposal to merge lp:~nbrinza/zorba/parse-fragment into
lp:~zorba-coders/zorba/zorba-experimental has been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~nbrinza/zorba/parse-fragment/+merge/89108
--
** Branch linked: lp:~zorba-coders/zorba/feature-transcode_streambuf
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/867159
Title:
fn:match fails if the string is non-utf8
Status in Zorba - The
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug905035/+merge/89099
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
Should the actual atomic type be present in the attribute element?
What's the use case for this? The type is declared in the index which should be
know by the user. It could be done but it's not clear how this would look like.
Do you want the QName of the atomic type
Review: Needs Information
Should we merge these changes?
--
https://code.launchpad.net/~zorba-coders/zorba/bug-896426/+merge/83885
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/http-client-version/+merge/88986
Your team Zorba Coders is subscribed to branch lp:zorba/http-client-module.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe :
It's not critical. I was hoping that Chris would have migrated more unit
tests.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-896426/+merge/83885
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
** Branch linked: lp:~paul-lucas/zorba/bug-907070
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/907070
Title:
Item API needs operator
Status in Zorba - The XQuery Processor:
New
Bug
77 matches
Mail list logo