Re: Storm 2.0.0 release?

2018-12-21 Thread Stig Rohde Døssing 
Looks good to me, the blockers list in JIRA is empty for 2.0.0.

Den fre. 21. dec. 2018 kl. 19.52 skrev Bobby Evans :

> I think all of the blockers are in now.  Please take a look and hopefully,
> we can get a release out soon.
>
> Thanks,
>
> Bobby
>
>
> On Fri, Dec 14, 2018 at 9:02 AM Bobby Evans  wrote:
>
> > Sorry I was out at a conference for the past week, and have been heads
> down
> > on a different project for a while before that.  I'll respond to the
> JIRA.
> > I am happy to let it go in.
> >
> > Thanks,
> >
> > Bobby
> >
> > On Thu, Dec 13, 2018 at 1:07 PM Stig Rohde Døssing <
> stigdoess...@gmail.com
> > >
> > wrote:
> >
> > > I think STORM-2990/3279 is ready. Bobby had a question (
> > > https://github.com/apache/storm/pull/2907#discussion_r234329136)
> > regarding
> > > whether Kafka offsets loop, but I wasn't sure where he was going with
> it,
> > > so I didn't want to merge prematurely.
> > >
> > > I agree that we can postpone STORM-2720. As far as I know it's waiting
> > for
> > > STORM-2990 to go in, since it's going to be touching the same code.
> > >
> > > Den tor. 13. dec. 2018 kl. 19.54 skrev Roshan Naik
> > > :
> > >
> > > >  Sounds like -  https://github.com/apache/storm/pull/2913
> (STORM-3290)
> > > is
> > > > merged -  https://github.com/apache/storm/pull/2908 (STORM-3276) is
> > > > nearly complete and may need some small tweaks. -
> > > > https://github.com/apache/storm/pull/2907  (STORM-2990, STORM-3279)
> > > > appears ready to be committed ?
> > > >
> > > > and- https://github.com/apache/storm/pull/2911   - (STORM-2720)
> > seems a
> > > > bit inactive and may not be critical enough to wait on.
> > > >
> > > > -roshan
> > > > On Monday, November 26, 2018, 9:49:30 AM PST, Stig Rohde Døssing
> <
> > > > stigdoess...@gmail.com> wrote:
> > > >
> > > >  I would like to get at least
> > https://github.com/apache/storm/pull/2913
> > > > (breaking changes) and https://github.com/apache/storm/pull/2908
> > > > (regression) in.
> > > >
> > > > I think it would be nice to also get
> > > > https://github.com/apache/storm/pull/2907 and
> > > > https://github.com/apache/storm/pull/2911 in, but if we're in a
> hurry
> > > they
> > > > could go in the next release.
> > > >
> > > > Den man. 26. nov. 2018 kl. 17.37 skrev Julien Nioche <
> > > > lists.digitalpeb...@gmail.com>:
> > > >
> > > > > Hi devs,
> > > > >
> > > > > Is there anything blocking the release of Storm 2.0? Any idea of
> when
> > > it
> > > > > could happen?
> > > > >
> > > > > Thanks
> > > > >
> > > > > Julien
> > > > >
> > > > > --
> > > > >
> > > > > *Open Source Solutions for Text Engineering*
> > > > >
> > > > > http://www.digitalpebble.com
> > > > > http://digitalpebble.blogspot.com/
> > > > > #digitalpebble 
> > > > >
> > > >
> > >
> >
>

Re: Storm 2.0.0 release?

2018-12-21 Thread Bobby Evans 
I think all of the blockers are in now.  Please take a look and hopefully,
we can get a release out soon.

Thanks,

Bobby


On Fri, Dec 14, 2018 at 9:02 AM Bobby Evans  wrote:

> Sorry I was out at a conference for the past week, and have been heads down
> on a different project for a while before that.  I'll respond to the JIRA.
> I am happy to let it go in.
>
> Thanks,
>
> Bobby
>
> On Thu, Dec 13, 2018 at 1:07 PM Stig Rohde Døssing  >
> wrote:
>
> > I think STORM-2990/3279 is ready. Bobby had a question (
> > https://github.com/apache/storm/pull/2907#discussion_r234329136)
> regarding
> > whether Kafka offsets loop, but I wasn't sure where he was going with it,
> > so I didn't want to merge prematurely.
> >
> > I agree that we can postpone STORM-2720. As far as I know it's waiting
> for
> > STORM-2990 to go in, since it's going to be touching the same code.
> >
> > Den tor. 13. dec. 2018 kl. 19.54 skrev Roshan Naik
> > :
> >
> > >  Sounds like -  https://github.com/apache/storm/pull/2913 (STORM-3290)
> > is
> > > merged -  https://github.com/apache/storm/pull/2908 (STORM-3276) is
> > > nearly complete and may need some small tweaks. -
> > > https://github.com/apache/storm/pull/2907  (STORM-2990, STORM-3279)
> > > appears ready to be committed ?
> > >
> > > and- https://github.com/apache/storm/pull/2911   - (STORM-2720)
> seems a
> > > bit inactive and may not be critical enough to wait on.
> > >
> > > -roshan
> > > On Monday, November 26, 2018, 9:49:30 AM PST, Stig Rohde Døssing <
> > > stigdoess...@gmail.com> wrote:
> > >
> > >  I would like to get at least
> https://github.com/apache/storm/pull/2913
> > > (breaking changes) and https://github.com/apache/storm/pull/2908
> > > (regression) in.
> > >
> > > I think it would be nice to also get
> > > https://github.com/apache/storm/pull/2907 and
> > > https://github.com/apache/storm/pull/2911 in, but if we're in a hurry
> > they
> > > could go in the next release.
> > >
> > > Den man. 26. nov. 2018 kl. 17.37 skrev Julien Nioche <
> > > lists.digitalpeb...@gmail.com>:
> > >
> > > > Hi devs,
> > > >
> > > > Is there anything blocking the release of Storm 2.0? Any idea of when
> > it
> > > > could happen?
> > > >
> > > > Thanks
> > > >
> > > > Julien
> > > >
> > > > --
> > > >
> > > > *Open Source Solutions for Text Engineering*
> > > >
> > > > http://www.digitalpebble.com
> > > > http://digitalpebble.blogspot.com/
> > > > #digitalpebble 
> > > >
> > >
> >
>

[GitHub] storm pull request #2908: STORM-3276: Updated Flux to deal with storm local ...

2018-12-21 Thread asfgit 
Github user asfgit closed the pull request at:

https://github.com/apache/storm/pull/2908


---

[GitHub] storm pull request #2929: MINOR: Correct comment about overflow limiting in ...

2018-12-21 Thread srdo 
GitHub user srdo opened a pull request:

https://github.com/apache/storm/pull/2929

MINOR: Correct comment about overflow limiting in JCQueue. Setting ov…

…erflow limit to 0 disables limiting, does not disable overflow

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/srdo/storm 
MINOR-jcqueue-overflow-limit-comment

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/storm/pull/2929.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2929


commit 7d14d02ed309fd724b3ecccf2a75a65b0359eee6
Author: Stig Rohde Døssing 
Date:   2018-12-21T17:51:17Z

MINOR: Correct comment about overflow limiting in JCQueue. Setting overflow 
limit to 0 disables limiting, does not disable overflow




---