[GitHub] storm issue #2636: 1.1.x branch - Add SSL functionality

2018-05-25 Thread cfriaszapater
Github user cfriaszapater commented on the issue:

https://github.com/apache/storm/pull/2636
  
Hi HeartSaVioR,

I saw your comment and understand the reason. Just that I haven't had time 
to do it yet, as I had problems building the full storm project master branch 
locally. I'll try to do that pull request in the future to master branch.

I did this to 1.1.x branch because we use it in our organization.

Regards.


---


[GitHub] storm pull request #2640: [STORM-3036] Add isRemoteBlobExists RPC interface ...

2018-05-25 Thread asfgit
Github user asfgit closed the pull request at:

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


---


Re: [VOTE] Release Apache Storm 1.2.2 (rc3)

2018-05-25 Thread Kishorkumar Patil
+1.
LGTM.

Kishor

On Wed, May 23, 2018 at 4:54 PM, Jungtaek Lim  wrote:

> REMINDER: The vote has been opened from 6 days ago, and still has only two
> binding +1s. Please verify the release candidate and vote.
>
> 2018년 5월 21일 (월) 오후 11:16, Julien Nioche 님이
> 작성:
>
> > +1 non binding
> >
> > compiled + ran StormCrawler topology without problems
> >
> > Thanks
> >
> > Julien
> >
> >
> >
> > On 17 May 2018 at 20:59, P. Taylor Goetz  wrote:
> >
> > > This is a call to vote on releasing Apache Storm 1.2.2 (rc3)
> > >
> > > Full list of changes in this release:
> > >
> > > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> > > 2.2-rc3/RELEASE_NOTES.html
> > >
> > > The tag/commit to be voted upon is v1.2.2:
> > >
> > > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=
> > >
> > d29eaa70280dd0ac4b7d393387fab4a91eee9e9d;hb=
> d2d6f40344e6cc92ab07f3a462d577
> > > ef6b61f8b1
> > >
> > > The source archive being voted upon can be found here:
> > >
> > > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> > > 2.2-rc3/apache-storm-1.2.2-src.tar.gz
> > >
> > > Other release files, signatures and digests can be found here:
> > >
> > > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.2-rc3/
> > >
> > > The release artifacts are signed with the following key:
> > >
> > > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=blob_
> > > plain;f=KEYS;hb=22b832708295fa2c15c4f3c70ac0d2bc6fded4bd
> > >
> > > The Nexus staging repository for this release is:
> > >
> > > https://repository.apache.org/content/repositories/orgapachestorm-1068
> > >
> > > Please vote on releasing this package as Apache Storm 1.2.2.
> > >
> > > When voting, please list the actions taken to verify the release.
> > >
> > > This vote will be open for 72 hours or until at least 3 PMC members
> vote
> > > +1.
> > >
> > > [ ] +1 Release this package as Apache Storm 1.2.2
> > > [ ]  0 No opinion
> > > [ ] -1 Do not release this package because...
> > >
> > > Thanks to everyone who contributed to this release.
> > >
> > > -Taylor
> > >
> >
> >
> >
> > --
> >
> > *Open Source Solutions for Text Engineering*
> >
> > http://www.digitalpebble.com
> > http://digitalpebble.blogspot.com/
> > #digitalpebble 
> >
>


Re: [VOTE] Release Apache Storm 1.1.3 (rc2)

2018-05-25 Thread Kishorkumar Patil
+1


Kishor

On Wed, May 23, 2018 at 4:54 PM, Jungtaek Lim  wrote:

> REMINDER: The vote has been opened from 5 days ago, and still has only one
> binding +1s. Please verify the release candidate and vote.
>
> 2018년 5월 19일 (토) 오전 10:49, Jungtaek Lim 님이 작성:
>
> > This release candidate contains md5/sha for src archives, unlike 1.2.2
> > RCs. May worth to check that.
> >
> > +1 (binding)
> >
> > On top to my verification on 1.2.3 rc1 (was +1),
> >
> > - verification on source/binary dist succeeded
> > - build succeeded on source
> > - spinning single cluster from binary dist succeeded
> > - ran RollingTopWords
> >
> > The difference between rc1 and rc2 is regarding storm-kafka-client patch,
> > so skipped checking storm-core features.
> >
> > Thanks,
> > Jungtaek Lim (HeartSaVioR)
> >
> > 2018년 5월 19일 (토) 오전 4:39, P. Taylor Goetz 님이 작성:
> >
> >> This is a call to vote on releasing Apache Storm 1.1.3 (rc2)
> >>
> >> Full list of changes in this release:
> >>
> >>
> >> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> 1.3-rc2/RELEASE_NOTES.html
> >>
> >> The tag/commit to be voted upon is v1.1.3:
> >>
> >>
> >> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=
> e983fef3d4dcd35a821cecb462e198505458d1a4;hb=c51c8fc251856291597a5007a07e43
> 3f26951464
> >>
> >> The source archive being voted upon can be found here:
> >>
> >>
> >> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> 1.3-rc2/apache-storm-1.1.3-src.tar.gz
> >>
> >> Other release files, signatures and digests can be found here:
> >>
> >> https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.1.3-rc2/
> >>
> >> The release artifacts are signed with the following key:
> >>
> >>
> >> https://git-wip-us.apache.org/repos/asf?p=storm.git;a=blob_
> plain;f=KEYS;hb=22b832708295fa2c15c4f3c70ac0d2bc6fded4bd
> >>
> >> The Nexus staging repository for this release is:
> >>
> >> https://repository.apache.org/content/repositories/orgapachestorm-1069
> >>
> >> Please vote on releasing this package as Apache Storm 1.1.3.
> >>
> >> When voting, please list the actions taken to verify the release.
> >>
> >> This vote will be open for 72 hours or until at least 3 PMC members vote
> >> +1.
> >>
> >> [ ] +1 Release this package as Apache Storm 1.1.3
> >> [ ]  0 No opinion
> >> [ ] -1 Do not release this package because...
> >>
> >> Thanks to everyone who contributed to this release.
> >>
> >> -Taylor
> >>
> >
>


Re: [VOTE] Release Apache Storm 1.2.2 (rc3)

2018-05-25 Thread Bobby Evans
+1 I built from the git tag, all the unit tests passed.  I ran a single
node cluster and everything worked for my small unit tests.

Thanks,

bobby

On Wed, May 23, 2018 at 3:54 PM Jungtaek Lim  wrote:

> REMINDER: The vote has been opened from 6 days ago, and still has only two
> binding +1s. Please verify the release candidate and vote.
>
> 2018년 5월 21일 (월) 오후 11:16, Julien Nioche 님이
> 작성:
>
> > +1 non binding
> >
> > compiled + ran StormCrawler topology without problems
> >
> > Thanks
> >
> > Julien
> >
> >
> >
> > On 17 May 2018 at 20:59, P. Taylor Goetz  wrote:
> >
> > > This is a call to vote on releasing Apache Storm 1.2.2 (rc3)
> > >
> > > Full list of changes in this release:
> > >
> > > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> > > 2.2-rc3/RELEASE_NOTES.html
> > >
> > > The tag/commit to be voted upon is v1.2.2:
> > >
> > > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=
> > >
> >
> d29eaa70280dd0ac4b7d393387fab4a91eee9e9d;hb=d2d6f40344e6cc92ab07f3a462d577
> > > ef6b61f8b1
> > >
> > > The source archive being voted upon can be found here:
> > >
> > > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
> > > 2.2-rc3/apache-storm-1.2.2-src.tar.gz
> > >
> > > Other release files, signatures and digests can be found here:
> > >
> > > https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.2-rc3/
> > >
> > > The release artifacts are signed with the following key:
> > >
> > > https://git-wip-us.apache.org/repos/asf?p=storm.git;a=blob_
> > > plain;f=KEYS;hb=22b832708295fa2c15c4f3c70ac0d2bc6fded4bd
> > >
> > > The Nexus staging repository for this release is:
> > >
> > > https://repository.apache.org/content/repositories/orgapachestorm-1068
> > >
> > > Please vote on releasing this package as Apache Storm 1.2.2.
> > >
> > > When voting, please list the actions taken to verify the release.
> > >
> > > This vote will be open for 72 hours or until at least 3 PMC members
> vote
> > > +1.
> > >
> > > [ ] +1 Release this package as Apache Storm 1.2.2
> > > [ ]  0 No opinion
> > > [ ] -1 Do not release this package because...
> > >
> > > Thanks to everyone who contributed to this release.
> > >
> > > -Taylor
> > >
> >
> >
> >
> > --
> >
> > *Open Source Solutions for Text Engineering*
> >
> > http://www.digitalpebble.com
> > http://digitalpebble.blogspot.com/
> > #digitalpebble 
> >
>


Re: [VOTE] Release Apache Storm 1.2.2 (rc3)

2018-05-25 Thread Ethan Li
+1

Built from the source.
Set up a single-node cluster and tested wordCountTopology 

-Ethan


> On May 25, 2018, at 10:44 AM, Bobby Evans  wrote:
> 
> +1 I built from the git tag, all the unit tests passed.  I ran a single
> node cluster and everything worked for my small unit tests.
> 
> Thanks,
> 
> bobby
> 
> On Wed, May 23, 2018 at 3:54 PM Jungtaek Lim  wrote:
> 
>> REMINDER: The vote has been opened from 6 days ago, and still has only two
>> binding +1s. Please verify the release candidate and vote.
>> 
>> 2018년 5월 21일 (월) 오후 11:16, Julien Nioche 님이
>> 작성:
>> 
>>> +1 non binding
>>> 
>>> compiled + ran StormCrawler topology without problems
>>> 
>>> Thanks
>>> 
>>> Julien
>>> 
>>> 
>>> 
>>> On 17 May 2018 at 20:59, P. Taylor Goetz  wrote:
>>> 
 This is a call to vote on releasing Apache Storm 1.2.2 (rc3)
 
 Full list of changes in this release:
 
 https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
 2.2-rc3/RELEASE_NOTES.html
 
 The tag/commit to be voted upon is v1.2.2:
 
 https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=
 
>>> 
>> d29eaa70280dd0ac4b7d393387fab4a91eee9e9d;hb=d2d6f40344e6cc92ab07f3a462d577
 ef6b61f8b1
 
 The source archive being voted upon can be found here:
 
 https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
 2.2-rc3/apache-storm-1.2.2-src.tar.gz
 
 Other release files, signatures and digests can be found here:
 
 https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.2.2-rc3/
 
 The release artifacts are signed with the following key:
 
 https://git-wip-us.apache.org/repos/asf?p=storm.git;a=blob_
 plain;f=KEYS;hb=22b832708295fa2c15c4f3c70ac0d2bc6fded4bd
 
 The Nexus staging repository for this release is:
 
 https://repository.apache.org/content/repositories/orgapachestorm-1068
 
 Please vote on releasing this package as Apache Storm 1.2.2.
 
 When voting, please list the actions taken to verify the release.
 
 This vote will be open for 72 hours or until at least 3 PMC members
>> vote
 +1.
 
 [ ] +1 Release this package as Apache Storm 1.2.2
 [ ]  0 No opinion
 [ ] -1 Do not release this package because...
 
 Thanks to everyone who contributed to this release.
 
 -Taylor
 
>>> 
>>> 
>>> 
>>> --
>>> 
>>> *Open Source Solutions for Text Engineering*
>>> 
>>> http://www.digitalpebble.com
>>> http://digitalpebble.blogspot.com/
>>> #digitalpebble 
>>> 
>> 



Re: [VOTE] Release Apache Storm 1.1.3 (rc2)

2018-05-25 Thread Ethan Li
+1

Built from the source.
Set up a single-node cluster and tested wordCountTopology 

-Ethan

> On May 25, 2018, at 10:59 AM, Kishorkumar Patil  
> wrote:
> 
> +1
> 
> 
> Kishor
> 
> On Wed, May 23, 2018 at 4:54 PM, Jungtaek Lim  wrote:
> 
>> REMINDER: The vote has been opened from 5 days ago, and still has only one
>> binding +1s. Please verify the release candidate and vote.
>> 
>> 2018년 5월 19일 (토) 오전 10:49, Jungtaek Lim 님이 작성:
>> 
>>> This release candidate contains md5/sha for src archives, unlike 1.2.2
>>> RCs. May worth to check that.
>>> 
>>> +1 (binding)
>>> 
>>> On top to my verification on 1.2.3 rc1 (was +1),
>>> 
>>> - verification on source/binary dist succeeded
>>> - build succeeded on source
>>> - spinning single cluster from binary dist succeeded
>>> - ran RollingTopWords
>>> 
>>> The difference between rc1 and rc2 is regarding storm-kafka-client patch,
>>> so skipped checking storm-core features.
>>> 
>>> Thanks,
>>> Jungtaek Lim (HeartSaVioR)
>>> 
>>> 2018년 5월 19일 (토) 오전 4:39, P. Taylor Goetz 님이 작성:
>>> 
 This is a call to vote on releasing Apache Storm 1.1.3 (rc2)
 
 Full list of changes in this release:
 
 
 https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
>> 1.3-rc2/RELEASE_NOTES.html
 
 The tag/commit to be voted upon is v1.1.3:
 
 
 https://git-wip-us.apache.org/repos/asf?p=storm.git;a=tree;h=
>> e983fef3d4dcd35a821cecb462e198505458d1a4;hb=c51c8fc251856291597a5007a07e43
>> 3f26951464
 
 The source archive being voted upon can be found here:
 
 
 https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.
>> 1.3-rc2/apache-storm-1.1.3-src.tar.gz
 
 Other release files, signatures and digests can be found here:
 
 https://dist.apache.org/repos/dist/dev/storm/apache-storm-1.1.3-rc2/
 
 The release artifacts are signed with the following key:
 
 
 https://git-wip-us.apache.org/repos/asf?p=storm.git;a=blob_
>> plain;f=KEYS;hb=22b832708295fa2c15c4f3c70ac0d2bc6fded4bd
 
 The Nexus staging repository for this release is:
 
 https://repository.apache.org/content/repositories/orgapachestorm-1069
 
 Please vote on releasing this package as Apache Storm 1.1.3.
 
 When voting, please list the actions taken to verify the release.
 
 This vote will be open for 72 hours or until at least 3 PMC members vote
 +1.
 
 [ ] +1 Release this package as Apache Storm 1.1.3
 [ ]  0 No opinion
 [ ] -1 Do not release this package because...
 
 Thanks to everyone who contributed to this release.
 
 -Taylor
 
>>> 
>> 



[GitHub] storm pull request #2694: STORM-3061: Upgrade version of hdrhistogram

2018-05-25 Thread revans2
GitHub user revans2 opened a pull request:

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

STORM-3061: Upgrade version of hdrhistogram

not too much is using hdrhistogram.

storm-metrics,
storm-loadgen,
storm-starter,
storm-elasticsearch,
storm-elasticsearch-examples

The ES code is not running because of what appear to be issues with the 
version of guava.  when we shade guava it should take care of that, and I will 
be able to test these changes better.

I think I am going to change focus for a while and put in shading so we can 
fix some of these issues and actually test these things.  I manually tested the 
others and the code works.

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

$ git pull https://github.com/revans2/incubator-storm STORM-3061-hdrhisto

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

https://github.com/apache/storm/pull/2694.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 #2694


commit 5e11cb99e94b6157b1aa7b97f0eaa57faa2d5df5
Author: Robert (Bobby) Evans 
Date:   2018-05-25T20:02:16Z

STORM-3061: Upgrade version of hdrhistogram




---