Project

General

Profile

Actions

Bug #2209

closed

arvados.api('v1').collections().list(limit=X).execute() returns ~half X results

Added by Tom Clegg about 10 years ago. Updated about 10 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
-
Category:
-
Story points:
1.0
Release:
Release relationship:
Auto

Subtasks 5 (0 open5 closed)

Task #2373: Figure out why, and fix. Start with paging branch.ClosedTom Clegg03/23/2014Actions
Task #2491: Reproduce "available > returned < limit" and write testResolvedTom Clegg03/23/2014Actions
Task #2410: Merge 2209 branch (it just adds some tests)ResolvedTom Clegg03/23/2014Actions
Task #2493: Fix inaccurate items_available current user has permission to collection via multiple groups/links.ResolvedTom Clegg03/23/2014Actions
Task #2494: Review 2209-correct-items-available branchResolvedTom Clegg03/23/2014Actions
Actions #1

Updated by Tom Clegg about 10 years ago

  • Story points set to 1.0
Actions #2

Updated by Tom Clegg about 10 years ago

  • Target version set to 2014-03-26 Debt service and dev painkillers
Actions #3

Updated by Tom Clegg about 10 years ago

  • Status changed from New to In Progress

Could not reproduce. (Fixed in #2075?)

Adding tests for limit and offset params instead.

Actions #4

Updated by Anonymous about 10 years ago

  • Status changed from In Progress to Resolved

Applied in changeset arvados|commit:8dd84d8f667aa40b9c8ce91d1a9248e1a7218039.

Actions #5

Updated by Tom Clegg about 10 years ago

  • Status changed from Resolved to New
Actions #6

Updated by Tom Clegg about 10 years ago

  • Target version changed from 2014-03-26 Debt service and dev painkillers to 2014-04-16 Dev tools and data/resource management
Actions #7

Updated by Tom Clegg about 10 years ago

Misha can still reproduce. With limit=10000, server returns ~3000 items but claims ~4000 items_available.

Actions #8

Updated by Brett Smith about 10 years ago

2209-correct-items-available looks like a nice clean branch to me. Just one thing: in the test, the assignment for unique_uuids doesn't seem to actually ensure uniqueness, as far as I can tell. Should it? That'd be easy to do with a set operator, like unique_uuids = [] | resp['items'].collect { |i| i['uuid'] }.compact.

Either way, I think this is good to merge.

Actions #9

Updated by Anonymous about 10 years ago

  • Status changed from New to Resolved
  • % Done changed from 78 to 100

Applied in changeset arvados|commit:5a19b986fd10f0584b33eb5953e72bd373e487d2.

Actions

Also available in: Atom PDF