- 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())
Review: Approve
Very nicely done feature. Approving... but there are some conflicts.
--
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-coders
Post to :
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/paging_index_probe-2012-09-18T16-33-49.228Z/log.html
--
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
Your team Zorba Coders is subscribed to branch lp:zorba.
--
The attempt to merge lp:~davidagraf/zorba/paging_index_probe 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 paging_index_probe-2012-09-18T16-33-49.228Z is
The proposal to merge lp:~davidagraf/zorba/paging_index_probe into lp:zorba has
been updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
--
The proposal to merge lp:~davidagraf/zorba/paging_index_probe into lp:zorba has
been updated.
Status: Needs review = Approved
For more details, see:
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
--
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/paging_index_probe-2012-09-18T17-17-56.517Z/log.html
--
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Validation queue job paging_index_probe-2012-09-18T17-17-56.517Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
The proposal to merge lp:~davidagraf/zorba/paging_index_probe into lp:zorba has
been updated.
Status: Approved = Merged
For more details, see:
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
--
- 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
- 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
Problems:
- Based on the paging.xqlib module (in the tests), the following query should
return the empty sequence:
idml:probe-index-range-value-skip(xs:QName(data:idx-age-range-val),
10, 2, 3, true(), true(), true(), true())
Fixed
But it returns all the items with age 3. This seems
Review: Needs Fixing
- 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)
- idml:probe-index-range-value-skip(xs:QName(data:idx-age-range-val), 2, 3,
true(), true(), true(), true())
reports
Review: Needs Fixing
Problems:
- Based on the paging.xqlib module (in the tests), the following query should
return the empty sequence:
idml:probe-index-range-value-skip(xs:QName(data:idx-age-range-val),
10, 2, 3, true(), true(), true(),
- idml:probe-index-range-value sorts the nodes implicitly in document order,
i.e. they are not returned in the order of the index keys. The skipping is
done before the sorting. This might be really confusing to the user.
Essentially, he doesn't have a way to know which items are skipped. We
Review: Approve
--
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
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 :
David Graf has proposed merging lp:~davidagraf/zorba/paging_index_probe into
lp:zorba.
Requested reviews:
David Graf (davidagraf)
For more details, see:
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
skip for value probes
--
The proposal to merge lp:~davidagraf/zorba/paging_index_probe into lp:zorba has
been updated.
Commit Message changed to:
skip for value probe functions
For more details, see:
https://code.launchpad.net/~davidagraf/zorba/paging_index_probe/+merge/122621
--
18 matches
Mail list logo
