Re: Pull requests for review

2017-07-16 Thread Jean-Baptiste Onofré
Hi Almas, as said before, you should take a look on the two links that I provided to you. You will see that you are a contributor, so a committer will do the review and the merge if the PR is correct. As reviewer, I will do it. Regards JB On 07/16/2017 07:30 PM, Apache Enthu wrote: Hi,

Re: Pull requests for review

2017-07-16 Thread Eugene Kirpichov
You will not be able to merge them: an Apache Beam committer has to do it for you. The people you specified as reviewers on the first two PRs (Kenn and JB) are definitely committers so they can do it. On Sun, Jul 16, 2017, 10:30 AM Apache Enthu wrote: > Hi, > > Am i fine

Re: Pull requests for review

2017-07-16 Thread Apache Enthu
Hi, Am i fine to merge these PR's? Thanks, Almas On Fri, Jul 14, 2017 at 10:28 PM, Apache Enthu wrote: > Hi, > > Could someone please review my PR's and let me know if any changes: > > https://github.com/apache/beam/pull/3526 > https://github.com/apache/beam/pull/3534 >

Pull requests for review

2017-07-14 Thread Apache Enthu
Hi, Could someone please review my PR's and let me know if any changes: https://github.com/apache/beam/pull/3526 https://github.com/apache/beam/pull/3534 https://github.com/apache/beam/pull/3535 Thanks, Almas