zorba-coders team mailing list archive
-
zorba-coders team
-
Mailing list archive
-
Message #14410
Re: [Merge] lp:~davidagraf/zorba/paging_index_probe into lp:zorba
> > - the probe-index-point-value-skip and probe-index-range-value-skip
> functions
> > in dml.xq don't declare the $skip parameters (but their documentation does)
> Fixed.
> >
> > - idml:probe-index-range-value-skip(xs:QName("data:idx-age-range-val"), 2,
> 3,
> > true(), true(), true(), true())
> > reports
> > '"data:idx-age-range-val": invalid number of arguments to index operation;
> > given 5 expected multiple of 6;'
> > but I'm counting 7 parameters
> Fixed (was not my fault :-) ). Unfortunately, when trying to write a test for
> this, I found a bug: https://bugs.launchpad.net/zorba/+bug/1051897 Can you
> please double check if it is really a bug?
I commented on the bug.
> > - It would be great if the iterator plan could mention if the iterator does
> > skipping and/or counting. For example, ./test/apitest -i paging.xq contains
> > the following snippet
> >
> > <ProbeIndexRangeValueIterator id="0x1a2fb50">
> > <SingletonIterator value="xs:QName(http://www.test.com/,data,idx-age-
> range-
> > val)" id="0x1a146a0"/>
> > <SingletonIterator value="xs:integer(7)" id="0x1a2c830"/>
> > <SingletonIterator value="xs:integer(2)" id="0x1a2c880"/>
> > <SingletonIterator value="xs:integer(3)" id="0x1a2c8d0"/>
> > <SingletonIterator value="xs:boolean(true)" id="0x1a2c920"/>
> > <SingletonIterator value="xs:boolean(true)" id="0x1a2c970"/>
> > <SingletonIterator value="xs:boolean(true)" id="0x1a2c9c0"/>
> > <SingletonIterator value="xs:boolean(true)" id="0x1a2ca10"/>
> > </ProbeIndexRangeValueIterator>
> >
> > for
> >
> > count(idml:probe-index-range-value-skip(xs:QName("data:idx-age-range-val"),
> > 7, 2, 3, true(), true(), true(), true()))
> Is it ok to create a bug entry for this? Because this has to be done in the
> collection iterator too. It can be assigned to me. But I would like to get
> those merge proposals thru.
Does it take long to do this. It would be really nice and I'm afraid it will not
get done if we create a bug for it. ;-)
- Should a negative skip value be an error? If not, some documentation is missing here.
- ChangeLog entry is missing.
--
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
Your team Zorba Coders is subscribed to branch lp:zorba.
Follow ups
References