Github user danny0405 commented on the issue:
https://github.com/apache/storm/pull/2800
@srdo
I think the only difference is that compared to `AtomicReference`,
`ConcurrentHashMap` can keep thread safe but can not ensure that the value read
is up to date, which will cause some
Github user zd-project commented on the issue:
https://github.com/apache/storm/pull/2800
Agreed. I think atomic reference is really just for Clojure compatibility
there. Iâll finish this up.
---
+1 for releasing 2.0.
May be the RC can be cut once critical patches are merged.
On Tue, 11 Sep 2018 at 10:28, Stig Rohde Døssing
wrote:
> +1 to cut an RC.
>
> Here are a couple of PRs that could maybe go in
>
> https://github.com/apache/storm/pull/2719
>
GitHub user revans2 opened a pull request:
https://github.com/apache/storm/pull/2826
STORM-3219: Update javascript to report errors consistently
Some of this was motivated for cases where nimbus was down the UI would not
report any errors. The goal here is to make sure that
Github user arunmahadevan closed the pull request at:
https://github.com/apache/storm/pull/2498
---
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2800
@zd-project I'd like to finish this up. Let me know if you want to make the
last couple of fixes, otherwise I'll open a new PR containing this fix.
@danny0405 I thought about it a bit more, and
GitHub user kishorvpatil opened a pull request:
https://github.com/apache/storm/pull/2825
[STORM-3220] Allow ability to enable/disable http binding
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kishorvpatil/incubator-storm
Github user arunmahadevan commented on the issue:
https://github.com/apache/storm/pull/2773
Seems to fix a few race conditions, may be we can take this forward.
---
Github user arunmahadevan commented on a diff in the pull request:
https://github.com/apache/storm/pull/2773#discussion_r216796216
--- Diff:
storm-server/src/main/java/org/apache/storm/blobstore/BlobStoreUtils.java ---
@@ -191,6 +192,8 @@ public static boolean
+1 to cut an RC.
Here are a couple of PRs that could maybe go in
https://github.com/apache/storm/pull/2719
https://github.com/apache/storm/pull/2800 (this one requires some changes,
but we should be able to fix it pretty quickly)
also would like to get https://github.com/apache/storm/pull/2805
GitHub user revans2 opened a pull request:
https://github.com/apache/storm/pull/2824
STORM-3218: Return the correct error from AuthorizedUserFilter
The changes to ReqContext are not needed. I started to make a change
there, and cleaned it up before realizing that it was not going
GitHub user Ethanlm opened a pull request:
https://github.com/apache/storm/pull/2823
[STORM-3206] validate topoConf only against Config.java during submission
Storm should only validate client-side conf when user submits a topology.
Otherwise settings like
We need your help to make the Apache Washington DC Roadshow on Dec 4th a
success.
What do we need most? Speakers!
We're bringing a unique DC flavor to this event by mixing Open Source
Software with talks about Apache projects as well as OSS CyberSecurity,
OSS in Government and and OSS Career
Hi all,
Any updates on this release?
On Wed, Aug 8, 2018 at 12:47 AM Stig Rohde Døssing
wrote:
> Hi devs,
>
> I've seen a couple of requests for releasing Storm 1.2.3 soon on the users
> list. I know we're currently working toward a release for Storm 2.0.0, but
> I think it would be nice to
14 matches
Mail list logo
