GoogleCloudPlatform / appengine-mapreduce

A library for running MapReduce jobs on App Engine
https://github.com/GoogleCloudPlatform/appengine-mapreduce/wiki/1-MapReduce
Apache License 2.0
234 stars 109 forks source link

DatastoreInputReader filter item in list #97

Open RubenMattan opened 8 years ago

RubenMattan commented 8 years ago

When you want to check if a list contains an item you normally check by doing .filter("list =", item) but DatastoreInputReader validation is expecting a list instead of an item.

Should be mapper_params = { "filters": [("list", "=", item)] } instead of mapper_params = { "filters": [("list", "=", [item])] }

Validate in source code

eko-wibowo commented 8 years ago

@RubenMattan I opened a pull request for it https://github.com/GoogleCloudPlatform/appengine-mapreduce/pull/94 this seems only affect db.Model as ndb is fine.