Re: beam-site issues with Jenkins and MergeBot

I'll take a look this afternoon Eugene, thanks! I'll also send a PR to update MergeBot documentation to make testing and deployment information clearer. On Thu, Aug 17, 2017 at 3:58 PM, Eugene Kirpichov wrote: > Hi Jason, > > Mergebot seems to be generally working, thanks!

Re: Style: how much testing for transform builder classes?

Hi all, This seems to have slipped through the cracks. I'd like to raise this again since I'm doing a cleanup in https://github.com/apache/beam/pull/3730 , and I'd like to get consensus and add the guidance to PTransform Style Guide or the Testing Guide. Let me rephrase my suggestions from the

Re: [VOTE] Release 2.1.0, release candidate #3

Hi JB, Any updates on finalizing the release? Thanks. On Thu, Aug 17, 2017 at 5:42 AM Aljoscha Krettek wrote: > (Belated) +1 > > * verified signatures > * verified that Quickstart works with Flink Runner > > > On 16. Aug 2017, at 20:41, Robert Bradshaw

Re: [VOTE] Release 2.1.0, release candidate #3

Hi I'm in vacation so I'm looking for a decent Internet connection to finalize the release. I keep you posted. Regards JB On Aug 18, 2017, 17:48, at 17:48, Eugene Kirpichov wrote: >Hi JB, > >Any updates on finalizing the release? > >Thanks. > >On Thu, Aug 17,

Re: Policy for stale PRs

bq. component leads regularly triage their components, including unassigning issues. +1 On Fri, Aug 18, 2017 at 5:11 PM, Ahmet Altay wrote: > To summarize the stale PR issue, do we agree on the following statement: > > A PR becomes stale after its author fails to

Proposal: file-based IOs should support readAllMatches()

Hi all, I've been adding new features to TextIO and AvroIO recently, see e.g. https://github.com/apache/beam/pull/3725. The features are: - withHintMatchesManyFiles() - readAll() that reads a PCollection of filepatterns - configurable treatment of filepatterns that match no files -

Re: Becoming a project contributor?

Thank you! On 2017-08-18 02:11, Kenneth Knowles wrote: > Done, and welcome! > > On Thu, Aug 17, 2017 at 2:24 PM, Asha Rostamianfar < > arost...@google.com.invalid> wrote: > > > Hi, > > > > I have started working on BEAM-2774 > >

Re: Proposal: file-based IOs should support readAllMatches()

+1 for this. Also it looks like IO authors should be able to use existing 'ReadAllViaFileBasedSource' transform when implementing FooIO.readAllMatches(). - Cham On Fri, Aug 18, 2017 at 2:38 PM Eugene Kirpichov wrote: > Hi all, > > I've been adding new features to

Re: Policy for stale PRs

To summarize the stale PR issue, do we agree on the following statement: A PR becomes stale after its author fails to respond to actionable comments for 60 days. The community will close stale PRs. Author is welcome to reopen the same PR again in the future. The associated JIRAs will be

Re: Style: how much testing for transform builder classes?

Hi Eugene It sounds good to me. Regards JB On Aug 18, 2017, 21:42, at 21:42, Eugene Kirpichov wrote: >Hi all, > >This seems to have slipped through the cracks. I'd like to raise this >again >since I'm doing a cleanup in https://github.com/apache/beam/pull/3730 ,

Re: Becoming a project contributor?

Done, and welcome! On Thu, Aug 17, 2017 at 2:24 PM, Asha Rostamianfar < arost...@google.com.invalid> wrote: > Hi, > > I have started working on BEAM-2774 > and was wondering how to > become a project contributor so that the issue can be assigned

Jenkins build is back to normal : beam_Release_NightlySnapshot #507

See