Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2872
---
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2871
---
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2877
Looks great, +1
---
Github user d2r commented on a diff in the pull request:
https://github.com/apache/storm/pull/2877#discussion_r224898629
--- Diff: integration-test/run-it.sh ---
@@ -91,6 +90,11 @@ function start_storm_process() {
echo starting: storm $1
sudo su storm -c "export
Github user d2r commented on a diff in the pull request:
https://github.com/apache/storm/pull/2877#discussion_r224892760
--- Diff: integration-test/run-it.sh ---
@@ -91,6 +90,11 @@ function start_storm_process() {
echo starting: storm $1
sudo su storm -c "export
Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2877#discussion_r224891745
--- Diff: integration-test/run-it.sh ---
@@ -91,6 +90,11 @@ function start_storm_process() {
echo starting: storm $1
sudo su storm -c "export
Github user d2r commented on the issue:
https://github.com/apache/storm/pull/2877
@srdo @revans2
This seemed to work on my personal repo.
I did not bother trying to set maximum client connections to `200` as we
did formerly, because we
GitHub user d2r opened a pull request:
https://github.com/apache/storm/pull/2877
STORM-3255: Uses dev-zookeeper in travis-ci
Instead of trying to install a zookeeper package that may or may not be
available, let's use the same ZooKeeper we ship with storm.
You can merge this pull
Agreed. We should deprecate repl and make STORM-3253 part of 2.0 RC.
-Kishor
On Fri, Oct 12, 2018 at 12:14 PM Derek Dagit
wrote:
> On Fri, Oct 12, 2018 at 9:53 AM Kishorkumar Patil >
> wrote:
>
> > We need to fix the possible leaking of slot in
> >
Github user kishorvpatil commented on the issue:
https://github.com/apache/storm/pull/2871
> @kishorvpatil I am not sure if we want to swallow IOException, since it
may be due to some serious problem which we want to propagate and nimbus cannot
continue in that case.
Agreed. We
On Fri, Oct 12, 2018 at 9:53 AM Kishorkumar Patil
wrote:
> We need to fix the possible leaking of slot in
> https://issues.apache.org/jira/browse/STORM-3254.
> The patch is upIhttps://github.com/apache/storm/pull/2876
> -Kishor
>
>
STORM-3253: Deprecates the storm repl subcommand #2875
We may
We need to fix the possible leaking of slot in
https://issues.apache.org/jira/browse/STORM-3254.
The patch is up https://github.com/apache/storm/pull/2876
-Kishor
On Thu, Oct 11, 2018 at 1:47 PM P. Taylor Goetz wrote:
> Canceling so blockers can be fixed for the 2.0.0 release.
>
> -Taylor
>
>
GitHub user kishorvpatil opened a pull request:
https://github.com/apache/storm/pull/2876
[STORM-3254] Don't wait for localization of blobs if assignments change.
Delay in blobstore can cause slot to become unusable.
You can merge this pull request into a Git repository by
13 matches
Mail list logo