add component tag to pull request title / commit comment

2016-05-09 Thread Manu Zhang
Guys,

As I've been developing Gearpump runner for Beam, I've closely watched the
pull requests updates for API changes. Sometimes, it turns out changes are
only to be applied to a specific runner after I go through the codes. Could
we add a tag in the pull request title / commit comment to mark whether
it's API change or Runner specific change, or something else, like,
[BEAM-XXX] [Direct-Java] Comments... ?

Thanks,
Manu Zhang


Re: Process / contribution guide

2016-05-09 Thread Jesse Anderson
I'd add some more information about checkstyle. That's the one that trips
me up. I haven't dealt with checkstyle before and the rules violations
output in mvn isn't very clear.

Right now it's discussed in the committer section, but it affects
contributors too.

I'd add some discussion of the common ones I hit like extra whitespace and
import order. I think the other ones were pretty clear. I'd also add the
mvn command to only run a checkstyle check. Also add the command to skip
the checkstyle (mvn clean install -Dcheckstyle.skip=true).

Thanks,

Jesse

On Sun, May 8, 2016 at 12:52 PM Jean-Baptiste Onofré 
wrote:

> Hi Davor,
>
> My bad: I did a cherry-pick instead of a merge on a PR.
>
> I will now be careful to apply a merge.
>
> Sorry about that.
>
> Regards
> JB
>
> On 05/08/2016 09:08 PM, Davor Bonaci wrote:
> > Hi everyone,
> > I wanted to send a quick remainder that we should all try to follow our
> own
> > contribution guide.
> >
> > Recently, there have been several cases where commits didn't go through
> the
> > pull requests / review, pull requests that were merge differently, not
> > closed automatically by tooling, etc.
> >
> > I'd kindly ask to try your best to follow our own process. That said, we
> > now have more experience in this type of development -- if there's any
> > point that should be re-discussed, please bring it up for consideration.
> >
> > Thanks!
> >
> > Davor
> >
>
> --
> Jean-Baptiste Onofré
> jbono...@apache.org
> http://blog.nanthrax.net
> Talend - http://www.talend.com
>


Re: ApacheCon next week

2016-05-09 Thread Ismaël Mejía
Hello,

If everything goes well I will be there  too, I hope to meet all the
present members of the beam dev community.

Ismaël Mejía

On Sun, May 8, 2016 at 8:22 PM, Davor Bonaci 
wrote:

> I'll be there Monday through either Wednesday or Thursday.
>
> On Sun, May 8, 2016 at 1:53 AM, Jean-Baptiste Onofré 
> wrote:
>
> > Hi Martin,
> >
> > yes, I will be at ApacheBigData & ApacheCon Tuesday, Wednesday, Thursday.
> >
> > I will land Monday evening and fly back to France on Thrusday night.
> >
> > Regards
> > JB
> >
> >
> > On 05/08/2016 09:59 AM, Martin Suchanek wrote:
> >
> >> Hi Davor, JB,
> >>
> >> Will you be at the "Apache: Big Data" (
> >> http://events.linuxfoundation.org/events/apache-big-data-north-america
> )
> >> conference as well, Monday through Wednesday?
> >>
> >> On Sat, May 7, 2016 at 10:37 PM Jean-Baptiste Onofré 
> >> wrote:
> >>
> >> Hi Davor,
> >>>
> >>> I will be there ;)
> >>>
> >>> I added a Beam session in the podlings shark tank:
> >>>
> >>> https://wiki.apache.org/apachecon/ACEU16PodlingSharkTank
> >>>
> >>>
> >>>
> >>>
> http://apachecon2016.sched.org/event/6OIN/podlings-shark-tank-roman-shaposhnik-pivotal?iframe=no=i:100;=yes=no
> >>>
> >>> @Davor, I count on you for this session with me ;)
> >>>
> >>> Regards
> >>> JB
> >>>
> >>> On 05/07/2016 10:18 PM, Davor Bonaci wrote:
> >>>
>  Hi everyone,
>  I wanted to check if any of the current or prospective Beam
> contributors
>  will be attending ApacheCon next week in Vancouver.
> 
>  If you'll be there and would like to talk about all-things-Beam,
> please
>  reach out!
> 
>  JB, the two of us will talk, of course. See you there ;)
> 
>  Thanks,
>  Davor
> 
> 
> >>> --
> >>> Jean-Baptiste Onofré
> >>> jbono...@apache.org
> >>> http://blog.nanthrax.net
> >>> Talend - http://www.talend.com
> >>>
> >>>
> >>
> > --
> > Jean-Baptiste Onofré
> > jbono...@apache.org
> > http://blog.nanthrax.net
> > Talend - http://www.talend.com
> >
>