New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 679064 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Add batching operations to luci/gae.

Project Member Reported by d...@chromium.org, Jan 6 2017

Issue description

Add batching operations to luci/gae. This involves:
1) Defining production constraint values and encoding them into luci/gae.
2) Adding the capability to transparently perform batch datastore Put and taskqueue Add.
3) Replacing "dsQueryBatch" filter with a service/datastore batching implementation.
 

Comment 1 by d...@chromium.org, Jan 6 2017

Cc: vadimsh@chromium.org iannucci@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/57a13888482ea38f957b9301a59b42e3a75ddea6

commit 57a13888482ea38f957b9301a59b42e3a75ddea6
Author: Dan Jacques <dnj@chromium.org>
Date: Sat Jan 07 01:56:41 2017

Roll luci-go and luci-gae.

This primarily picks up the new batching functonality in luci/gae.

infra/go/src/github.com/luci/luci-go:
fb0dd9bf Remove dsQueryBatch in favor of ds.Batcher.
77c4eac9 Fix BigTable multi-entry row cached Tail.

infra/go/src/github.com/luci/gae:
a05a77f Fix improper use of Stop in datastore.
c68b2ca Add batch Add to service/taskqueue.
82198d4 Add datastore Put to service/datastore.Batcher.
e987b6b Add ds.Batcher class, implement Run, GetAll.

TBR=vadimsh@chromium.org
BUG= chromium:679064 
TEST=None

Change-Id: I8e3168c2b9be77a83d00106debe7593e58da0180
Reviewed-on: https://chromium-review.googlesource.com/425835
Reviewed-by: Daniel Jacques <dnj@chromium.org>
Commit-Queue: Daniel Jacques <dnj@chromium.org>

[modify] https://crrev.com/57a13888482ea38f957b9301a59b42e3a75ddea6/DEPS

Comment 5 by d...@chromium.org, Jan 7 2017

Status: Fixed (was: Started)

Sign in to add a comment