GitHub user omkreddy opened a pull request:
https://github.com/apache/storm/pull/2099
STORM-2482: [WIP] Auto populate Hive Credentials
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/omkreddy/storm AutoHiveNewBranch
Alternative
Thanks to everyone that voted.
Of the 10 people that voted 8 voted for google as their first choice. 1 voted
for Hadoop and 1 voted for a modified form of Google.
With that I think google wins. I will start by merging in the checkstyle pull
request for google
https://github.com/apache/storm/pu
Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2093
Still +1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the fe
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2093
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enab
Github user harshach commented on the issue:
https://github.com/apache/storm/pull/2097
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user SreeramGarlapati commented on a diff in the pull request:
https://github.com/apache/storm/pull/2098#discussion_r114821502
--- Diff:
external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/BinaryEventDataSystemPropertiesScheme.java
---
@@ -0,0 +1,51 @@
Github user SreeramGarlapati commented on a diff in the pull request:
https://github.com/apache/storm/pull/2098#discussion_r114825627
--- Diff:
external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/BinaryEventDataSystemPropertiesScheme.java
---
@@ -0,0 +1,51 @@
Github user SreeramGarlapati commented on a diff in the pull request:
https://github.com/apache/storm/pull/2098#discussion_r114826039
--- Diff:
external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/FieldConstants.java
---
@@ -23,5 +23,6 @@
public static fi
Github user ryanpersaud commented on the issue:
https://github.com/apache/storm/pull/1044
Now you've piqued my curiosity - what is it that you don't like about
rotation actions?
Writers are only stored in the writers map, so once they are removed, there
is no way for FSDataOu
Github user knusbaum commented on the issue:
https://github.com/apache/storm/pull/2096
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user knusbaum commented on the issue:
https://github.com/apache/storm/pull/2078
This looks good to me, but I'm pretty sure we've frozen features on
1.0.x-branch. It would be better to put it on 1.x-branch, and needs to go into
master for 2.x as well.
---
If your project is se
Github user rban1 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2098#discussion_r114857534
--- Diff:
external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/BinaryEventDataSystemPropertiesScheme.java
---
@@ -0,0 +1,51 @@
+package
Github user rban1 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2098#discussion_r114857665
--- Diff:
external/storm-eventhubs/src/main/java/org/apache/storm/eventhubs/spout/FieldConstants.java
---
@@ -23,5 +23,6 @@
public static final String
GitHub user adityasharad opened a pull request:
https://github.com/apache/storm/pull/2100
STORM-2503: Fix lgtm.com alerts on equality and comparison operations.
This PR addresses STORM-2503, by fixing a number of alerts found by
lgtm.com at https://lgtm.com/projects/g/apache/storm/a
Github user erikdw commented on a diff in the pull request:
https://github.com/apache/storm/pull/2094#discussion_r114897599
--- Diff: docs/Setting-up-a-Storm-cluster.md ---
@@ -102,9 +102,9 @@ The time to allow any given healthcheck script to run
before it is marked failed
sto
Github user erikdw commented on a diff in the pull request:
https://github.com/apache/storm/pull/2094#discussion_r114898011
--- Diff: docs/Setting-up-a-Storm-cluster.md ---
@@ -102,9 +102,9 @@ The time to allow any given healthcheck script to run
before it is marked failed
sto
16 matches
Mail list logo