Implement support for converting db values to datastore values with db.Query.filter

Implement support for converting db values to datastore values with db.Query.filter

Closes #37

R=sgjesse@google.com

Committed: https://github.com/dart-lang/gcloud/commit/a7bd666544542e529a5d9d02601b48bb4166f0f1

[kustermann, 2015-12-11 11:08:44]

kustermann@google.com changed reviewers:
+ sgjesse@google.com

[kustermann, 2015-12-11 11:08:44]

https://codereview.chromium.org/1520863002/diff/1/test/db/e2e/db_test_impl.dart
File test/db/e2e/db_test_impl.dart (right):

https://codereview.chromium.org/1520863002/diff/1/test/db/e2e/db_test_impl.da...
test/db/e2e/db_test_impl.dart:478: query,
usersSortedNameDescNicknameAsc.length);
There is still some flakiness on the buildbot. I decided to make it more robust
and make all the query tests use exponential backoff. Hopefully that removes
this last bit of flakiness.

(The reason why I belief that the [waitUntilEntitiesReady] is not enough is that
the backend has different tables for Entities, OnePropertyIndex,
ComposedPropertyIndex, so even if the first table is ready, the other two are
still eventually consistent)

[Søren Gjesse, 2015-12-14 07:53:24]

lgtm

https://codereview.chromium.org/1520863002/diff/1/test/db/e2e/db_test_impl.dart
File test/db/e2e/db_test_impl.dart (right):

https://codereview.chromium.org/1520863002/diff/1/test/db/e2e/db_test_impl.da...
test/db/e2e/db_test_impl.dart:523: var models = await
runQueryWithExponentialBackoff(query, fooUsers.length);
Long line - and a few more below.

https://codereview.chromium.org/1520863002/diff/1/test/db/e2e/db_test_impl.da...
test/db/e2e/db_test_impl.dart:614: Future<List<db.Model>>
runQueryWithExponentialBackoff(
This looks like a good idea.

[kustermann, 2015-12-14 09:28:32]

Description was changed from

==========
Implement support for converting db values to datastore values with
db.Query.filter

Closes #37
==========

to

==========
Implement support for converting db values to datastore values with
db.Query.filter

Closes #37

R=sgjesse@google.com

Committed:
https://github.com/dart-lang/gcloud/commit/a7bd666544542e529a5d9d02601b48bb41...
==========

[kustermann, 2015-12-14 09:28:33]

Committed patchset #2 (id:20001) manually as
a7bd666544542e529a5d9d02601b48bb4166f0f1 (presubmit successful).

[kustermann, 2015-12-14 09:41:16]

Thanks

https://codereview.chromium.org/1520863002/diff/1/test/db/e2e/db_test_impl.dart
File test/db/e2e/db_test_impl.dart (right):

https://codereview.chromium.org/1520863002/diff/1/test/db/e2e/db_test_impl.da...
test/db/e2e/db_test_impl.dart:523: var models = await
runQueryWithExponentialBackoff(query, fooUsers.length);
On 2015/12/14 07:53:24, Søren Gjesse wrote:
> Long line - and a few more below.

Done.