The proposal to merge lp:~markos-za/zorba/markos-bugs into lp:zorba has been
updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/78881
--
https://code.launchpad.net/~markos-za/zorba/markos-bugs/+merge/78881
Your tea
** Changed in: zorba
Status: New => Fix Released
--
You received this bug notification because you are a member of Zorba
Coders, which is the registrant for Zorba.
https://bugs.launchpad.net/bugs/867262
Title:
Ext func arg item seq can not be consumed 2 times
Status in Zorba - The XQue
> Hi Markos,
>
> i think that the flag "haveListType" is not reset correctly if the element
> passes from
>
> haveTypedValue && haveTypedTypedValue && haveListType
> to
> haveTypedValue && haveTypedTypedValue && !haveListType
>
> I would add:
> else
>textChild->resetHaveListValue();
> near l
The proposal to merge lp:~markos-za/zorba/bugs2 into lp:zorba has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~markos-za/zorba/bugs2/+merge/78834
--
https://code.launchpad.net/~markos-za/zorba/bugs2/+merge/78834
Your team Zorba Coders is s
Review: Approve
--
https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/78814
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
The proposal to merge lp:~fcavalieri/zorba/bugs into lp:zorba has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/78814
--
https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/78814
Your team Zorba Coders is s
> No reason to reject but why do we rename functions in the store api? It will
> cause 28msec some effort when merging the changes.
Because getNodeReference is more descriptive and more consistent/symmetric with
the node-reference function, and the getNodeByReference method.
But if you think it
Review: Approve
--
https://code.launchpad.net/~markos-za/zorba/bugs2/+merge/78834
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
The proposal to merge lp:~markos-za/zorba/bugs2 into lp:zorba has been updated.
Status: Approved => Work in progress
For more details, see:
https://code.launchpad.net/~markos-za/zorba/bugs2/+merge/78834
--
https://code.launchpad.net/~markos-za/zorba/bugs2/+merge/78834
Your team Zorba Coders
Review: Approve
--
https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/79070
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
The proposal to merge lp:~fcavalieri/zorba/bugs into lp:zorba has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/79070
--
https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/79070
Your team Zorba Coders is s
Review: Approve
--
https://code.launchpad.net/~fcavalieri/zorba/bugs2/+merge/78401
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
Review: Approve
--
https://code.launchpad.net/~fcavalieri/zorba/setsatundo/+merge/79093
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
> > > > function name too long? node-structural-information
> > >
> > > Any suggestions?
> > >
> > > node-information
> > > node-position
> > > node-uri
> > > structural-information
> > I like node-information as name for both - the module and the function.
> > That's at least consistent with the n
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79187
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-c
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/updrevalidate
into lp:zorba.
Requested reviews:
Federico Cavalieri (fcavalieri)
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79331
--
https
Federico, I forgot to submit some cosmetic changes to this branch before
pushing the "approve" button. So, now I have created this new merge proposal.
Can you please review? Thanks!.
--
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79331
Your team Zorba Coders is subscribed
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79331
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-c
The proposal to merge lp:~markos-za/zorba/indexes into lp:zorba has been
updated.
Commit Message changed to:
1. Big reorg of the code for general indexes:
1.1 Unified code for creating hash- and tree-based index
1.2 Unified code among different kinds of probe functions
1.3 Improved performance b
Matthias, I think I am done with the coding part. I am still working on the
documentation, so that part is not finished yet. But feel free to start looking
at the code as soon as you like. Since this is a rather big reorg, I am afraid
that diffs will not help very much. You just have to look at
The proposal to merge lp:~markos-za/zorba/indexes into lp:zorba has been
updated.
Commit Message changed to:
1. Big reorg of the code for general indexes:
1.1 Unified code for creating hash- and tree-based index
1.2 Unified code among different kinds of probe functions
1.3 Improved performance b
> Why variable numUpdates in PULImpl::mergeUpdateList is declared, assigned but
> never read?
Good question :) The easy answer is to just remove it. But I think it can
actually be used to replace the myList.size() in the 2 for loops at lines 1289
and 1308. Even though new entries can be added in
The proposal to merge lp:~zorba-coders/zorba/updrevalidate into lp:zorba has
been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge/79331
--
https://code.launchpad.net/~zorba-coders/zorba/updrevalidate/+merge
> Renamed the module and function to node-position.
> How should I replace "structural information" in the functions comments?
> "node position uri"?
I rewrote somewhat the introductory comment in node-position.xq. Here is the
new version:
(:~
: This module provides a function (np:node-position
Federico, I don't think it is necessary to put the position uris into the uri
pool because they are not likely to be shared by multiple data items so you
won't really be saving much space (you may be actually end-up wasting space
because the uri pool storage has some overhead by itself). Do you
Should StructuralAnyUriItem::isDescendant() check that the "other" node is not
an attribute?
--
https://code.launchpad.net/~zorba-coders/zorba/structuralrelationships2/+merge/78395
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Pos
> > Should StructuralAnyUriItem::isDescendant() check that the "other" node is
> not
> > an attribute?
>
> Yes, you are right! Descendant doesn't include attributes.
>
> May I add a in-subtree-of function with the same semantics as the current
> descendant?
>
> Thanks
> Federico
It's ok with m
> Hi Markos,
> i noticed that the parse uri function now uses strtoul without checking errno
> with the following effects:
>
> A missing component is considered 0:
> np:in-same-tree-of(xs:anyURI("zorba:..1.50"),xs:anyURI("zorba:0.0.1.50"))
> ->true
>
>
> Out of ulong value-space == ULONG_MAX
> n
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos1 into
lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos1/+merge/79723
optimization: store
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos1/+merge/79723
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/markos1 into lp:zorba has been
updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos1/+merge/79723
--
https://code.launchpad.net/~zorba-coders/zorba/markos1/+merge/79723
Your team Z
Review: Approve
--
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/79732
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
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos1 into
lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos1/+merge/79746
Memory optimization
Matthias, this is not really ready for checkin yet. We need to find a way to
address the xray regression caused by this memory optimization.
--
https://code.launchpad.net/~zorba-coders/zorba/markos1/+merge/79746
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://l
Review: Approve
--
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/79743
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
The proposal to merge lp:~matthias-brantner/zorba/bug-fixing into lp:zorba has
been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/+merge/79743
--
https://code.launchpad.net/~matthias-brantner/zorba/bug-fixing/
> Connection dropped...
>
> Do you know why document nodes have a component less than any other kind of
> node when they are the root of their tree?
>
I finally found out what you meant (the code is in the OrdPathNode ctor, not
the OrdPath ctor).
I think the reason why element node as root has a
Federico, I think the documentation in node-position.xq needs some more work. I
can do this, but first we must agree on what is exactly the functionality and
the assumptions about node positions that you rely on. For example, in the
documentation of the np:node-position() function, you write:
"
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug_872732/+merge/79272
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-code
Juan, the getExternalVariables method should be part of the static context, not
the dynamic context.
Probably the easiest way to implement this is to add an "externalOnly" param to
the static_context::getVariables method and then add a getExternalVariables
method in StaticContextImpl, which wil
Carlos, let's try to finis this today and commit to the main trunk. So, please
add a test for w3c collections, and also update the ChangeLog file with a
one-line description of the change.
--
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
Your team Zorba Coders i
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
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/~z
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
--
https://code.launchpad.net/~zorba-coders/zorba/collecti
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
--
https://code.launchpad.net/~zorba-coders/zorba/collecti
Ok, it looks better now, but we are not done yet
First, the comment in include/zorba/static_context.h for the
getExternalVariables() method is wrong. The method returns the qnames of all
the external variables that are in-scope within that static context. It's not
about whether the variable
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
--
https://code.launchpad.net/~zorba-coders/zorba/collecti
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
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/~z
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
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/~z
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
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/~z
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
--
https://code.launchpad.net/~zorba-coders/zorba/collecti
> > Federico, I think the documentation in node-position.xq needs some more
> work.
> > I can do this, but first we must agree on what is exactly the functionality
> > and the assumptions about node positions that you rely on. For example, in
> the
> > documentation of the np:node-position() functi
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
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/~z
The proposal to merge lp:~zorba-coders/zorba/collection-counting into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/collection-counting/+merge/79765
--
https://code.launchpad.net/~zorba-coders/zorba/collecti
> > Second, to get the external variables from all the modules that participate
> in
> > a query, we also need a getExternalVariables() at the XQueryImpl level. This
> > method will go through all the static context objs in
> > theCompilerCB->theSctxMap and compute the union of the external variabl
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos2 into
lp:zorba.
Requested reviews:
Matthias Brantner (matthias-brantner)
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos2/+merge/80129
Groupby optimization
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos2/+merge/80129
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/markos2 into lp:zorba has been
updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos2/+merge/80129
--
https://code.launchpad.net/~zorba-coders/zorba/markos2/+merge/80129
Your team Z
Review: Approve
--
https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/80487
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
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/structuralrelationships2/+merge/78395
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.n
Markos Zaharioudakis has proposed merging lp:~zorba-coders/zorba/markos-skinny
into lp:zorba.
Requested reviews:
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-skinny/+merge/80756
moved base uri and doc uri from the XmlTree to
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-skinny/+merge/80756
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-c
The proposal to merge lp:~zorba-coders/zorba/markos-skinny into lp:zorba has
been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-skinny/+merge/80756
--
https://code.launchpad.net/~zorba-coders/zorba/markos-skinny/+merge
The proposal to merge lp:~zorba-coders/zorba/markos-skinny into lp:zorba has
been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-skinny/+merge/80756
--
https://code.launchpad.net/~zorba-coders/zorba/markos-skinny/+merge
Markos Zaharioudakis has proposed merging
lp:~zorba-coders/zorba/markos-annotations into lp:zorba.
Requested reviews:
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-annotations/+merge/80899
simplified and optimized the
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-annotations/+merge/80899
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/~zo
The proposal to merge lp:~zorba-coders/zorba/markos-annotations into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-annotations/+merge/80899
--
https://code.launchpad.net/~zorba-coders/zorba/markos-ann
The proposal to merge lp:~zorba-coders/zorba/markos-annotations into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-annotations/+merge/80899
--
https://code.launchpad.net/~zorba-coders/zorba/markos-ann
Juan, can you please make zorba-coders be the owner of this branch. I want to
make some changes to it, but I cannot because it currently belongs to you only.
--
https://code.launchpad.net/~juan457/zorba/xqxq-api-changes/+merge/79589
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Ma
> some parameters of the probe-index-range-general#7 function are not
> documented.
> Also, should this function be variadic?
I have added the missing params.
No, the function is not variadic because it applies to general indexes only,
whicg have only one key column.
--
https://code.launchpad.n
The proposal to merge lp:~zorba-coders/zorba/markos-annotations into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/markos-annotations/+merge/80899
--
https://code.launchpad.net/~zorba-coders/zorba/markos-ann
> Done just changed the ownership to zorba-coders
>
> > Juan, can you please make zorba-coders be the owner of this branch. I want
> to
> > make some changes to it, but I cannot because it currently belongs to you
> > only.
Hmm, I couldn't find it. what is the new name of the branch?
--
https://
> The implementation of getExternalVariables in the xqueryimpl level is done.
Juan, I fixed the implementation of the XQuery::getExternalVariables() method.
The main problem there was that you were comparing QName items using their
string value. This is not correct because the string value of a
Review: Approve
--
https://code.launchpad.net/~markos-za/zorba/indexes/+merge/79334
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
Mor
The proposal to merge lp:~markos-za/zorba/indexes into lp:zorba has been
updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~markos-za/zorba/indexes/+merge/79334
--
https://code.launchpad.net/~markos-za/zorba/indexes/+merge/79334
Your team Zorba Code
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/structuralrelationships2/+merge/81053
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.n
The proposal to merge lp:~zorba-coders/zorba/structuralrelationships2 into
lp:zorba has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/structuralrelationships2/+merge/81053
--
https://code.launchpad.net/~zorba-coders/zorb
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/81432
trying to fix Windows compilation problem
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/81432
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-
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/81432
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+me
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/81434
fixed a wrong assertion in annotations.cpp
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/81434
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-
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/81434
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+me
Here are a few problems I can see so far:
1. I don't think we can do function caching for variadic functions. The current
implementation of user_function::computeResultCaching certainly does not
support variadic functions, but more importantly, we need a fixed number of
params to form the index
> Found a function that does exactly the same changing it.
>
> > I implemented the isBoundExternalVariable adding a new function in
> > context/dynamic_context called is_set_variable which returns true if it is
> set
> > or false if it doesn't I could have use the get_variable but I would have to
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/81857
optimizer optimization:avoid double traversal
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/81857
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+me
> Added the error throw and also changed getExternalVariables to use iterators
> instead of a vector
Juan, XPST0008 is not a bad choice for the error code, but in this case, the
error does not come from an xquery program but from the zorba c++ api. So, you
must create a new error code. To do thi
Markos Zaharioudakis has proposed merging
lp:~zorba-coders/zorba/substring-intopt into lp:zorba.
Requested reviews:
Markos Zaharioudakis (markos-za)
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/substring-intopt/+merge/81949
Optimization of the fn:substring function
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/81857
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-
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/81857
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+me
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/82026
modified serializer so that it does not have to
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/82026
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-
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/82026
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+me
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/82045
Fixed bug when parsing a document with a base
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/82045
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-
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/82045
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+me
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/82074
Fixed bug when parsing a document with a base
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+merge/82074
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-
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/82074
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+me
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/82074
--
https://code.launchpad.net/~zorba-coders/zorba/markos-scratch/+me
601 - 700 of 1832 matches
Mail list logo
