[Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
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/90210 -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/testsuites_update-2012-01-26T08-00-00.711Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
Running validation queue again with XQFTTS enabled (forgot last time). -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
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:274 (message): Validation queue job testsuites_update-2012-01-26T08-00-00.711Z is finished. The final status was: 2 tests did not succeed - changes not commited. Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
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/90210 -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 921954] [NEW] Typo in geo module documentation
Public bug reported: There is a small typo in the geo module documentation: "err:XQDY0027 if any of the nodes passed in the $lat-long-degress parameter is not valid according to the schema." It should be: "err:XQDY0027 if any of the nodes passed in the $lat_long_degrees parameter is not valid according to the schema." ** Affects: zorba Importance: Low Status: New ** Changed in: zorba Importance: Undecided => Low ** Changed in: zorba Milestone: None => 2.2 -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/921954 Title: Typo in geo module documentation Status in Zorba - The XQuery Processor: New Bug description: There is a small typo in the geo module documentation: "err:XQDY0027 if any of the nodes passed in the $lat-long-degress parameter is not valid according to the schema." It should be: "err:XQDY0027 if any of the nodes passed in the $lat_long_degrees parameter is not valid according to the schema." To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/921954/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
What I've figured out is the following: 1. The additional regressions and things I see are, I presume, the results of the newer version of XQTS. This was changed by Sorin in r10594. I thought that revision was already merged on the trunk, but evidently not. 2. The 6 regressions I reported above are ones that we've hit in the past - they appear to be failing on the validation queue machine, but not on most developer machines. In the past, we have more or less decided this is because the validation queue machine is running an older version of ICU that has certain bugs. So, I have updated both known_failures files to add those 6 tests back in, and am approving this merge. -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
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/90210 -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/testsuites_update-2012-01-26T09-03-19.173Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 921954] Re: Typo in geo module documentation
In geoprj there is a typo in "The purpose of this module is to provide convertion" ->"The purpose of this module is to provide conversion" -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/921954 Title: Typo in geo module documentation Status in Zorba - The XQuery Processor: New Bug description: There is a small typo in the geo module documentation: "err:XQDY0027 if any of the nodes passed in the $lat-long-degress parameter is not valid according to the schema." It should be: "err:XQDY0027 if any of the nodes passed in the $lat_long_degrees parameter is not valid according to the schema." To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/921954/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
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:274 (message): Validation queue job testsuites_update-2012-01-26T09-03-19.173Z is finished. The final status was: 2 tests did not succeed - changes not commited. Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
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/90210 -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
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/90210 -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
Something went weird with my commit; trying again. -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/testsuites_update-2012-01-26T09-43-05.268Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
Validation queue job testsuites_update-2012-01-26T09-43-05.268Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/doxygen into lp:zorba
William Candillon has proposed merging lp:~zorba-coders/zorba/doxygen into lp:zorba. Requested reviews: Cezar Andrei (cezar-andrei) Matthias Brantner (matthias-brantner) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/90249 - Fix a couple of external links - Fix main documentation structure - Fix a couple of typos -- https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/90249 Your team Zorba Coders is subscribed to branch lp:zorba. === modified file 'doc/c/manual/indexpage.dox' --- doc/c/manual/indexpage.dox 2011-08-26 23:36:24 + +++ doc/c/manual/indexpage.dox 2012-01-26 10:15:43 + @@ -23,7 +23,7 @@ written in C. - To start with, this documentation provides a set of examples that demonstrate + To start with, this documentation provides a set of examples that demonstrate some features. */ === modified file 'doc/zorba/build.dox' --- doc/zorba/build.dox 2011-09-23 10:54:01 + +++ doc/zorba/build.dox 2012-01-26 10:15:43 + @@ -151,7 +151,7 @@ you need a working Visual Studio (Express) installation (2008, 2010). - If you want to compile Zorba using Visual Studio 2008, you will need to install the - http://www.microsoft.com/downloads/en/details.aspx?FamilyId=D466226B-8DAB-445F-A7B4-448B326C48E7&displaylang=en"; target="_blank">Visual C++ 2008 Feature Pack Release. + http://www.microsoft.com/downloads/en/details.aspx?FamilyId=D466226B-8DAB-445F-A7B4-448B326C48E7"; target="_blank">Visual C++ 2008 Feature Pack Release. - Make sure that the libraries required for Zorba are built with the same version of Visual Studio that you use. - The Windows version of some required packages @@ -233,10 +233,10 @@ a set of public keys freely available on the internet. - On Windows, Zorba expects this file to be called \c cacert.pem - and searches for it in current directory and system paths. + and searches for it in the current directory and system paths. - This \c cacert.pem file can be found on the curl page: http://curl.haxx.se/docs/caextract.html . -- In order to disable the validation of server certificate, +- In order to disable the validation of the server certificate, the \c ZORBA_VERIFY_PEER_SSL_CERTIFICATE can be set to \c OFF in cmake (and actually this is the default value). === modified file 'doc/zorba/build_noncore.dox' --- doc/zorba/build_noncore.dox 2011-09-23 10:54:01 + +++ doc/zorba/build_noncore.dox 2012-01-26 10:15:43 + @@ -77,7 +77,7 @@ \subsubsection cclientunix Unix/Linux/Mac OS X Notes: - There are some known issues with the CClient packages - that come with diffrent Linux distributions. + that come with different Linux distributions. - On x32 bit OpenSuse and also on x64 bit Ubuntu, we noticed that the CClient shared library is broken (undefined symbol: mm_dlog). === modified file 'doc/zorba/commandline.dox' --- doc/zorba/commandline.dox 2011-11-04 11:31:13 + +++ doc/zorba/commandline.dox 2012-01-26 10:15:43 + @@ -1,18 +1,5 @@ /** \page commandline Command Line Utility - - Table of Contents - - \ref commandLineUsage - \ref commandLineHelp - \ref commandlineexamples - - \ref cmdlineExample1 - \ref cmdlineExample2 - - - - \section commandLineUsage Usage %zorba [options] [-q Inline_Query ...] or \n %zorba [options] -f [-q Query_File ...] === modified file 'doc/zorba/data_lifecycle.dox' --- doc/zorba/data_lifecycle.dox 2011-09-08 19:44:12 + +++ doc/zorba/data_lifecycle.dox 2012-01-26 10:15:43 + @@ -123,7 +123,7 @@ Also please note that a data container that is available in the store will be available to all XQuery programs that are being executed synchronously. Again, please read more about the Zorba Data Stores below about data consistency details. -\section dl_examples Examples +\section dlexamples Examples In the following, we show a couple of examples to demonstrate how data can be retrieved and store in various kinds of containers. It is important to note that most of the examples uses the XQuery Scripting Extension for apply pending updates in order to make them visible to subsequent expressions in the same program. @@ -284,7 +284,7 @@ The reason is simple: the XQuery processor can understand the data flow and data lifecycle in the first case, while it cannot in the second. -\section dl_zorba_store Zorba Stores +\section dlzorbastore Zorba Stores Zorba defines a Store API that allows developers to seamlessly process XML data stored in different places. Essentially, the Store API is a C++ interface for === modified file 'doc/zorba/external_functions.dox' --- doc/zorba/external_functions.dox 2011-11-04 11:36:35 + +++ doc/zorba/external_functions.dox 2012-01-26 10:15:43 + @@ -111,7 +111,7 @@ be necessary for you to place the shared object such that Zorba can load it at runtime. -To locate a dynamic libary, Zorba first transforms its target +To locate a dynamic
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/testsuites_update into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/testsuites_update into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 -- https://code.launchpad.net/~zorba-coders/zorba/testsuites_update/+merge/90210 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 918157] Re: Add XQFTTS to validation queue
** Changed in: zorba Status: In Progress => Fix Committed -- 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: Fix Committed Bug description: 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 XQueryX and XQFTTS. These also will need to be resolved and merged for XQFTTS to be enabled on the queue. To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/918157/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 918211] Re: xqueryx fulltext w3c conformance generation
** Changed in: zorba Status: In Progress => Fix Committed -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/918211 Title: xqueryx fulltext w3c conformance generation Status in Zorba - The XQuery Processor: Fix Committed Bug description: 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 To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/918211/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 918626] Re: Debugger: Add unit tests
I have committed the test framework for debugger server that tests the DBGP communication in the lp:~zorba-coders/zorba/debugger_tests branch. It runs perfectly on Windows. On Linux it does NOT due to a lock that generates a dead lock (a condition still waiting). This happens only when a DebuggerTest assert fails. For example in asserting the message, the fileuri attribute is not correct on UNIX-based versions (Bug #918720). This assert failing causes some conditions to wait and the UnitTests runs forever. (This might be similar to a bug I have filed about another lock problem on non-WIndows platform: Bug #918720.) If all asserts succeed, UnitTests terminates correctly, but it is not acceptable. If, in case of a debugger unit test failure on Linux, UnitTests will block the remote queue on linux. As of today, my contribution to Zorba will be very limited (if at all) so I am removing myself as assignee from this bug. ** Changed in: zorba Status: In Progress => Opinion ** Changed in: zorba Assignee: Gabriel Petrovay (gabipetrovay) => Matthias Brantner (matthias-brantner) -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba. https://bugs.launchpad.net/bugs/918626 Title: Debugger: Add unit tests Status in Zorba - The XQuery Processor: Opinion Bug description: New debugger unit tests are needed with the introduction of the new debugger in Zorba 2.2. To manage notifications about this bug go to: https://bugs.launchpad.net/zorba/+bug/918626/+subscriptions -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/doxygen into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/doxygen/+merge/90249 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/stack into lp:~zorba-coders/zorba/zorba-experimental
The proposal to merge lp:~zorba-coders/zorba/stack into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/stack/+merge/90314 -- https://code.launchpad.net/~zorba-coders/zorba/stack/+merge/90314 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/stack into lp:~zorba-coders/zorba/zorba-experimental. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/stack into lp:~zorba-coders/zorba/zorba-experimental
The attempt to merge lp:~zorba-coders/zorba/stack into lp:~zorba-coders/zorba/zorba-experimental failed. Below is the output from the failed tests. experimental build started for lp:~zorba-coders/zorba/stack. Check out http://stack.zorba-xquery.com/ in half an hour or so. Logfile at http://zorbatest.lambda.nu:8080/remotequeue/experimental/stack.log . -- https://code.launchpad.net/~zorba-coders/zorba/stack/+merge/90314 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/stack into lp:~zorba-coders/zorba/zorba-experimental. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/stack into lp:~zorba-coders/zorba/zorba-experimental
The proposal to merge lp:~zorba-coders/zorba/stack into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/stack/+merge/90314 -- https://code.launchpad.net/~zorba-coders/zorba/stack/+merge/90314 Your team Zorba Coders is requested to review the proposed merge of lp:~zorba-coders/zorba/stack into lp:~zorba-coders/zorba/zorba-experimental. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 920717] Re: json 2.0 bug: json:serialize(json:parse(VALID_JSON))
I investigated this issue a little more. The problem seems to be in the
optimizer.
zorba -q tmp.xq -f --optimization-level O0
returns the correct result.
{"html":"bold"}
whereas
zorba -q tmp.xq -f --optimization-level O1
raises the error.
Tracing this further, it seems as if the json:parse function returns the
options element created in json:serialize (if the optimizer is used). It
clearly returns the correct value if the query is used without
json:serialize(...).
I'm assigning this to Markos to help looking into this. If I made a
guess, I would say that the problem is in the no-copy rule.
** Changed in: zorba
Assignee: Paul J. Lucas (paul-lucas) => Markos Zaharioudakis (markos-za)
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/920717
Title:
json 2.0 bug: json:serialize(json:parse(VALID_JSON))
Status in Zorba - The XQuery Processor:
New
Bug description:
Should be possible to call json:serialize(json:parse(VALID_JSON))
where VALID_JSON is any valid JSON string
see added failing test test/rbkt/zorba/json/json-snelson-serialize-parse:
import module namespace
json="http://www.zorba-xquery.com/modules/converters/json";;
declare variable $json-value := '{"html":"bold"}';
json:serialize(json:parse($json-value))
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/920717/+subscriptions
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~nbrinza/zorba/parse-fragment into lp:zorba
Review: Approve -- 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 : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/markos-scratch into lp:zorba
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/90336
type-related optimizagtions of boolean iterators
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/90336
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'ChangeLog'
--- ChangeLog 2012-01-25 21:20:30 +
+++ ChangeLog 2012-01-26 21:45:33 +
@@ -10,7 +10,8 @@
* Fixed bug 917923 (bug in copying outer var values into the eval dynamic context)
* Fixed bug 909126 (bug in cloning of var_expr)
* Fixed bug in destruction of exit_catcher_expr
- * Types-related optimization for the comparison operators
+ * Types-related optimizations for runtime operators (comparisons, FnBoolean, Or, And,
+Compare).
* Fixed bug #911585 (management of variables during eval)
* Fixed bug #866423 (fn:empty and fn:exists iterators must reset their input in
case of early-out)
=== modified file 'src/api/item.cpp'
--- src/api/item.cpp 2012-01-18 18:09:34 +
+++ src/api/item.cpp 2012-01-26 21:45:33 +
@@ -32,6 +32,7 @@
#include "api/unmarshaller.h"
#include "store/api/item.h"
+#include "store/api/item_factory.h"
#include "store/api/store.h"
#include "store/api/iterator.h"
#include "store/api/collection.h"
@@ -264,7 +265,12 @@
SYNC_CODE(AutoLock lock(GENV_STORE.getGlobalLock(), Lock::READ);)
-return &*m_item->getEBV();
+bool value = m_item->getEBV();
+
+store::Item_t result;
+GENV_ITEMFACTORY->createBoolean(result, value);
+
+return result.getp();
ITEM_CATCH
return Item();
=== modified file 'src/compiler/rewriter/rules/fold_rules.cpp'
--- src/compiler/rewriter/rules/fold_rules.cpp 2011-11-22 13:28:23 +
+++ src/compiler/rewriter/rules/fold_rules.cpp 2012-01-26 21:45:33 +
@@ -840,7 +840,7 @@
if ((constArg = dynamic_cast(arg)) != NULL)
{
- if (constArg->get_val()->getEBV()->getBooleanValue() == shortcircuit_val)
+ if (constArg->get_val()->getEBV() == shortcircuit_val)
return new const_expr(fo->get_sctx(), LOC(fo), (xs_boolean)shortcircuit_val);
}
else
=== modified file 'src/functions/func_booleans_impl.cpp'
--- src/functions/func_booleans_impl.cpp 2012-01-23 10:06:24 +
+++ src/functions/func_booleans_impl.cpp 2012-01-26 21:45:33 +
@@ -874,7 +874,6 @@
GENV_TYPESYSTEM.ANY_ATOMIC_TYPE_STAR,
GENV_TYPESYSTEM.ANY_ATOMIC_TYPE_STAR,
GENV_TYPESYSTEM.BOOLEAN_TYPE_ONE));
-// end Atomic Values Equivalent
}
=== modified file 'src/runtime/booleans/BooleanImpl.cpp'
--- src/runtime/booleans/BooleanImpl.cpp 2012-01-24 11:24:56 +
+++ src/runtime/booleans/BooleanImpl.cpp 2012-01-26 21:45:33 +
@@ -89,7 +89,7 @@
const PlanIterator* iter,
bool negate)
{
- store::Item_t item, temp;
+ store::Item_t item;
bool result;
bool is_sequence;
@@ -107,38 +107,41 @@
}
else
{
-xqtref_t type = tm->create_value_type(item);
-is_sequence = consumeNext(temp, iter, planState);
+store::SchemaTypeCode type = item->getTypeCode();
+
+store::Item_t item2;
+is_sequence = consumeNext(item2, iter, planState);
+
if (!is_sequence
&&
-(TypeOps::is_equal(tm, *type, *GENV_TYPESYSTEM.BOOLEAN_TYPE_ONE)
- || TypeOps::is_subtype(tm, *type, *GENV_TYPESYSTEM.STRING_TYPE_ONE)
- || TypeOps::is_subtype(tm, *type, *GENV_TYPESYSTEM.ANY_URI_TYPE_ONE)
- || TypeOps::is_subtype(tm, *type, *GENV_TYPESYSTEM.UNTYPED_ATOMIC_TYPE_ONE)
- || TypeOps::is_numeric(tm, *type)))
+(type == store::XS_BOOLEAN ||
+ TypeOps::is_subtype(type, store::XS_STRING) ||
+ TypeOps::is_subtype(type, store::XS_ANY_URI) ||
+ type == store::XS_UNTYPED_ATOMIC ||
+ TypeOps::is_numeric(type)))
{
// atomic type xs_boolean, xs_string, xs_anyURI, xs_untypedAtomic
// => effective boolean value is defined in the items
- temp = item->getEBV();
- result = negate ? (negate ^ temp->getBooleanValue()) : temp->getBooleanValue();
+ bool temp = item->getEBV();
+ result = negate ? (negate ^ temp) : temp;
}
else
{
if (is_sequence)
-throw XQUERY_EXCEPTION(
- err::FORG0006,
- ERROR_PARAMS(
-ZED( BadArgTypeForFn_2o34o ), "", "fn:boolean",
-ZED( EBVNotDefSeq_5 ), *type
- ),
- ERROR_LOC( loc )
-);
+ {
+xqtref_t type = tm->create_value_type(item);
+
+RAISE_ERROR(err::FORG0006, loc,
+ERROR_PARAMS(ZED(BadArgTypeForFn_2o34o),
+ "", "fn:boolean",
+ ZED(EBVNotDefSeq_5),
+ *type));
+ }
else
-throw XQUERY_EXCEPTION(
- err::FORG0006,
- ERROR_PARAMS( ZED( BadArgTypeForFn_2o34o ), "", "fn:boolean" ),
-
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/markos-scratch into lp:zorba
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/90336 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/markos-scratch into lp:zorba
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/90336 -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/90336 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/markos-scratch into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/markos-scratch-2012-01-26T21-49-04.992Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/90336 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/markos-scratch into lp:zorba
Validation queue job markos-scratch-2012-01-26T21-49-04.992Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/90336 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/markos-scratch into lp:zorba
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/90336 -- https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/90336 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 More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Bug 922432] [NEW] member accessor on object raises XPTY0004
Public bug reported:
The following query raises [err:XPTY0004]: ~NoTypePromotion_json
let $obj :=
{
"glossary": {
"title": "example glossary",
"GlossDiv": {
"title": "S",
"GlossList": {
"GlossEntry": {
"ID": "SGML",
"SortAs": "SGML",
"GlossTerm": "Standard Generalized Markup Language",
"Acronym": "SGML",
"Abbrev": "ISO 8879:1986",
"GlossDef": {
"para": "A meta-markup language, used to create
markup languages such as DocBook.",
"GlossSeeAlso": ["GML", "XML"]
},
"GlossSee": "markup"
}
}
}
}
}
return $obj("glossary")("title")
** Affects: zorba
Importance: High
Assignee: Markos Zaharioudakis (markos-za)
Status: New
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/922432
Title:
member accessor on object raises XPTY0004
Status in Zorba - The XQuery Processor:
New
Bug description:
The following query raises [err:XPTY0004]: ~NoTypePromotion_json
let $obj :=
{
"glossary": {
"title": "example glossary",
"GlossDiv": {
"title": "S",
"GlossList": {
"GlossEntry": {
"ID": "SGML",
"SortAs": "SGML",
"GlossTerm": "Standard Generalized Markup Language",
"Acronym": "SGML",
"Abbrev": "ISO 8879:1986",
"GlossDef": {
"para": "A meta-markup language, used to create
markup languages such as DocBook.",
"GlossSeeAlso": ["GML", "XML"]
},
"GlossSee": "markup"
}
}
}
}
}
return $obj("glossary")("title")
To manage notifications about this bug go to:
https://bugs.launchpad.net/zorba/+bug/922432/+subscriptions
--
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental
retry building the web interface with jsoniq support -- https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/90370 Your team Zorba Coders is requested to review the proposed merge of lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental
The proposal to merge lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/90370 -- https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/90370 Your team Zorba Coders is requested to review the proposed merge of lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental
The proposal to merge lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental has been updated. Commit Message changed to: just another try For more details, see: https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/90370 -- https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/90370 Your team Zorba Coders is requested to review the proposed merge of lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental
The attempt to merge lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental failed. Below is the output from the failed tests. experimental build started for lp:zorba/3.0. Check out http://3.0.zorba-xquery.com/ in half an hour or so. Logfile at http://zorbatest.lambda.nu:8080/remotequeue/experimental/3.0.log . -- https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/90370 Your team Zorba Coders is requested to review the proposed merge of lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental
The proposal to merge lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/90370 -- https://code.launchpad.net/~zorba-coders/zorba/jsoniq/+merge/90370 Your team Zorba Coders is requested to review the proposed merge of lp:zorba/3.0 into lp:~zorba-coders/zorba/zorba-experimental. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-896426 into lp:zorba
Review: Approve -- 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 Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-896426 into lp:zorba
The proposal to merge lp:~zorba-coders/zorba/bug-896426 into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-896426/+merge/83885 -- 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 Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-896426 into lp:zorba
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-896426-2012-01-27T07-38-56.145Z/log.html -- 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 Unsubscribe : https://launchpad.net/~zorba-coders More help : https://help.launchpad.net/ListHelp
