Re: Storm 2.0.0 release?

2018-12-13 Thread Stig Rohde Døssing
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-13 Thread 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 
 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 issue #2925: STORM-3302: Ensures we close streams to HDFS

2018-12-13 Thread kishorvpatil
Github user kishorvpatil commented on the issue:

https://github.com/apache/storm/pull/2925
  
@d2r  Thank you for the fix and squashing the commits. 


---