[ARVADOS] updated: 866eda506d29e407caec78a11c2719773246bfc1

git at public.curoverse.com git at public.curoverse.com
Thu Jan 14 15:21:41 EST 2016


Summary of changes:
 apps/workbench/app/models/arvados_resource_list.rb | 5 ++++-
 doc/sdk/cli/reference.html.textile.liquid          | 2 +-
 sdk/python/arvados/util.py                         | 2 ++
 services/api/config/application.default.yml        | 6 ++++++
 services/api/lib/load_param.rb                     | 6 ++----
 5 files changed, 15 insertions(+), 6 deletions(-)

       via  866eda506d29e407caec78a11c2719773246bfc1 (commit)
       via  3fb0f46498300eca6a97db88d76c33f855f5166c (commit)
       via  4487166336dd8521e7e1ea74f2990ec5f210a55c (commit)
       via  68967945ac9dfb524e196bcc2a4c259d115cdb6a (commit)
       via  f918d79484a5e355c9aae1dfc6622e6fe7c96f7f (commit)
      from  7d6ff766d02ddc486cb61be3a0bb9ce12e7a29ff (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 866eda506d29e407caec78a11c2719773246bfc1
Author: Tom Clegg <tom at curoverse.com>
Date:   Thu Jan 14 15:20:46 2016 -0500

    Fix cli "update" example. No issue #

diff --git a/doc/sdk/cli/reference.html.textile.liquid b/doc/sdk/cli/reference.html.textile.liquid
index 5e5f238..ef9882f 100644
--- a/doc/sdk/cli/reference.html.textile.liquid
+++ b/doc/sdk/cli/reference.html.textile.liquid
@@ -29,7 +29,7 @@ Get entire record for some specific user
 @arv user get --uuid 6dnxa-tpzed-iimd25zhzh84gbk@
 
 Update user record
- at arv user update --uuid 6dnxa-tpzed-iimd25zhzh84gbk --first-name "Bob"@
+ at arv user update --uuid 6dnxa-tpzed-iimd25zhzh84gbk --user '{"first_name":"Bob"}'@
 
 Get list of groups
 @arv group list@

commit 3fb0f46498300eca6a97db88d76c33f855f5166c
Merge: 7d6ff76 4487166
Author: Tom Clegg <tom at curoverse.com>
Date:   Thu Jan 14 15:19:48 2016 -0500

    Merge branch '8189-handle-large-collections-better' refs #8189


commit 4487166336dd8521e7e1ea74f2990ec5f210a55c
Author: Tom Clegg <tom at curoverse.com>
Date:   Thu Jan 14 13:15:08 2016 -0500

    8189: Use large page size only when fetching all pages.

diff --git a/apps/workbench/app/models/arvados_resource_list.rb b/apps/workbench/app/models/arvados_resource_list.rb
index 1be06d8..2706970 100644
--- a/apps/workbench/app/models/arvados_resource_list.rb
+++ b/apps/workbench/app/models/arvados_resource_list.rb
@@ -184,8 +184,10 @@ class ArvadosResourceList
     api_params[:order] = @orderby_spec if @orderby_spec
     api_params[:filters] = @filters if @filters
     api_params[:distinct] = @distinct if @distinct
-    # Default limit to (effectively) api server's MAX_LIMIT
-    api_params[:limit] = 2**(0.size*8 - 1) - 1
+    if @fetch_multiple_pages
+      # Default limit to (effectively) api server's MAX_LIMIT
+      api_params[:limit] = 2**(0.size*8 - 1) - 1
+    end
 
     item_count = 0
     offset = @offset || 0

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list