Add GetNoTxn to txnBuf as a hack.

Add GetNoTxn to txnBuf as a hack.

I really don't like this but short of fixing #23, I don't think there's much we
can do.

R=dnj@chromium.org, nodir@chromium.org
BUG=

Committed: https://github.com/luci/gae/commit/5787dea3cf12f40b4fc210c724dea92c7f5ff2bf

[dnj (Google), 2016-06-02 04:10:27]

dnj@google.com changed reviewers:
+ dnj@google.com

[dnj (Google), 2016-06-02 04:10:28]

lgtm w/ nit

https://codereview.chromium.org/2036593002/diff/1/filter/txnBuf/context.go
File filter/txnBuf/context.go (right):

https://codereview.chromium.org/2036593002/diff/1/filter/txnBuf/context.go#ne...
filter/txnBuf/context.go:51: // currently buffered transaction, but it also does
not see any effects of it.
nit: "currently-buffered"

https://codereview.chromium.org/2036593002/diff/1/filter/txnBuf/context.go#ne...
filter/txnBuf/context.go:59: c = context.WithValue(c, dsTxnBufHaveLock, nil)
Havelock? http://expanse.wikia.com/wiki/Dmitri_Havelock_(TV)

[iannucci, 2016-06-02 22:32:08]

The CQ bit was checked by iannucci@chromium.org

[iannucci, 2016-06-02 22:32:09]

The patchset sent to the CQ was uploaded after l-g-t-m from dnj@google.com
Link to the patchset: https://codereview.chromium.org/2036593002/#ps20001
(title: "fix nit")

[commit-bot: I haz the power, 2016-06-02 22:32:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2036593002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2036593002/20001

[iannucci, 2016-06-02 22:32:21]

https://chromiumcodereview.appspot.com/2036593002/diff/1/filter/txnBuf/contex...
File filter/txnBuf/context.go (right):

https://chromiumcodereview.appspot.com/2036593002/diff/1/filter/txnBuf/contex...
filter/txnBuf/context.go:51: // currently buffered transaction, but it also does
not see any effects of it.
On 2016/06/02 04:10:27, dnj (Google) wrote:
> nit: "currently-buffered"

Done.

[commit-bot: I haz the power, 2016-06-02 22:35:26]

Description was changed from

==========
Add GetNoTxn to txnBuf as a hack.

I really don't like this but short of fixing #23, I don't think there's much we
can do.

R=dnj@chromium.org, nodir@chromium.org
BUG=
==========

to

==========
Add GetNoTxn to txnBuf as a hack.

I really don't like this but short of fixing #23, I don't think there's much we
can do.

R=dnj@chromium.org, nodir@chromium.org
BUG=

Committed:
https://github.com/luci/gae/commit/5787dea3cf12f40b4fc210c724dea92c7f5ff2bf
==========

[commit-bot: I haz the power, 2016-06-02 22:35:28]

Committed patchset #2 (id:20001) as
https://github.com/luci/gae/commit/5787dea3cf12f40b4fc210c724dea92c7f5ff2bf