Re: Review Request 46465: Move Samza to use pull requests

2016-06-23 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46465/#review139320 --- Still reviewing.. samza_merge_pr.py (line 401)

[GitHub] samza pull request #13: removing git pull unsupported line from README

2016-06-23 Thread navina
Github user navina closed the pull request at: https://github.com/apache/samza/pull/13 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enable

Re: [GitHub] samza pull request #13: removing git pull unsupported line from README

2016-06-23 Thread Navina Ramesh
Please ignore. This was test! On Thu, Jun 23, 2016 at 5:43 PM, navina wrote: > GitHub user navina opened a pull request: > > https://github.com/apache/samza/pull/13 > > removing git pull unsupported line from README > > Testing pull requests! > > You can merge this pull request into

[GitHub] samza pull request #13: removing git pull unsupported line from README

2016-06-23 Thread navina
GitHub user navina opened a pull request: https://github.com/apache/samza/pull/13 removing git pull unsupported line from README Testing pull requests! You can merge this pull request into a Git repository by running: $ git pull https://github.com/navina/samza test-branch Alt

Re: Review Request 49138: SAMZA-936: Add double serde as build-in default

2016-06-23 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49138/#review139299 --- Ship it! lgtm! +1 - Navina Ramesh On June 23, 2016, 9:26 p.m

Re: Review Request 49138: SAMZA-889: Change log not working properly with In memory Store

2016-06-23 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49138/ --- (Updated June 23, 2016, 9:26 p.m.) Review request for samza. Changes ---

Re: Review Request 49138: SAMZA-936: Add double serde as build-in default

2016-06-23 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49138/ --- (Updated June 23, 2016, 9:26 p.m.) Review request for samza. Summary (updated

Re: Review Request 49138: SAMZA-889: Change log not working properly with In memory Store

2016-06-23 Thread Navina Ramesh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49138/#review139296 --- samza-api/src/main/java/org/apache/samza/storage/StorageEngine.ja

Review Request 49138: SAMZA-889: Change log not working properly with In memory Store

2016-06-23 Thread Yi Pan (Data Infrastructure)
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49138/ --- Review request for samza. Bugs: SAMZA-936 https://issues.apache.org/jira/br