[GitHub] storm pull request: [STORM-855] Add tuple batching

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/694#issuecomment-133200328 Sure, please take your time! --- 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

[jira] [Commented] (STORM-855) Add tuple batching

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705897#comment-14705897 ] ASF GitHub Bot commented on STORM-855: -- Github user HeartSaVioR commented on the pull

[jira] [Commented] (STORM-855) Add tuple batching

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705889#comment-14705889 ] ASF GitHub Bot commented on STORM-855: -- Github user mjsax commented on the pull reques

[GitHub] storm pull request: [STORM-855] Add tuple batching

2015-08-20 Thread mjsax
Github user mjsax commented on the pull request: https://github.com/apache/storm/pull/694#issuecomment-133198490 Thanks for your guidance! I will have a look at #521 and add some results. May take a few days... --- If your project is set up for it, you can reply to this email and hav

[jira] [Commented] (STORM-855) Add tuple batching

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705886#comment-14705886 ] ASF GitHub Bot commented on STORM-855: -- Github user HeartSaVioR commented on the pull

[GitHub] storm pull request: [STORM-855] Add tuple batching

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/694#issuecomment-133197501 No, there's no specific approach. You can refer @d2r's approach, https://github.com/apache/storm/pull/521#issuecomment-106038982 I think we would be inte

[jira] [Commented] (STORM-855) Add tuple batching

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705873#comment-14705873 ] ASF GitHub Bot commented on STORM-855: -- Github user mjsax commented on the pull reques

[GitHub] storm pull request: UserGroupInformation should use the same confi...

2015-08-20 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/storm/pull/471#issuecomment-133194879 @onlychoice you need to package hbase-site.xml along with the topology jar. Did you try doing that. --- If your project is set up for it, you can reply to this email an

[GitHub] storm pull request: [STORM-855] Add tuple batching

2015-08-20 Thread mjsax
Github user mjsax commented on the pull request: https://github.com/apache/storm/pull/694#issuecomment-133194835 Sure. Is there any specific approach I should take? How to add/report those result? I did a simple test already, using `ExclamationTopology` example. Setting batch size to

[jira] [Commented] (STORM-855) Add tuple batching

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705871#comment-14705871 ] ASF GitHub Bot commented on STORM-855: -- Github user HeartSaVioR commented on the pull

[GitHub] storm pull request: [STORM-855] Add tuple batching

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/694#issuecomment-133194709 FYI, you can use https://github.com/yahoo/storm-perf-test for benchmarking if you don't have your own. You need to modify pom.xml to let it points recent version o

[jira] [Commented] (STORM-855) Add tuple batching

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705861#comment-14705861 ] ASF GitHub Bot commented on STORM-855: -- Github user HeartSaVioR commented on the pull

[GitHub] storm pull request: [STORM-855] Add tuple batching

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/694#issuecomment-133193429 @mjsax I'm not familiar with clojure, too. Many committers will take a look, so don't worry. :) Before taking a look, I think you're encouraged to do ben

[jira] [Commented] (STORM-855) Add tuple batching

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705827#comment-14705827 ] ASF GitHub Bot commented on STORM-855: -- Github user mjsax commented on the pull reques

[GitHub] storm pull request: [STORM-855] Add tuple batching

2015-08-20 Thread mjsax
Github user mjsax commented on the pull request: https://github.com/apache/storm/pull/694#issuecomment-133187814 Great! Just a heads up: This is my first PR for Storm and I just started to learn Clojure. Please review very carefully. Looking forward to your feedback. --- If your

[GitHub] storm pull request: [STORM-855] Add tuple batching

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/694#issuecomment-133187367 asfbot recognizes your change and links. :) --- 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 pr

[jira] [Commented] (STORM-855) Add tuple batching

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705821#comment-14705821 ] ASF GitHub Bot commented on STORM-855: -- Github user HeartSaVioR commented on the pull

[jira] [Commented] (STORM-855) Add tuple batching

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-855?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705818#comment-14705818 ] ASF GitHub Bot commented on STORM-855: -- Github user mjsax commented on the pull reques

[GitHub] storm pull request: [STORM-855] Add tuple batching

2015-08-20 Thread mjsax
Github user mjsax commented on the pull request: https://github.com/apache/storm/pull/694#issuecomment-133186983 I guess I need to close an re-open this PR to get the linkage to JIRA... --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] storm pull request: [STROM-855] Add tuple batching

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/694#issuecomment-133186028 Could you fix typo in title, to let asfgit matches PR and JIRA issue properly? Thanks! --- If your project is set up for it, you can reply to this email and have you

[GitHub] storm pull request: [STROM-855] Add tuple batching

2015-08-20 Thread mjsax
GitHub user mjsax opened a pull request: https://github.com/apache/storm/pull/694 [STROM-855] Add tuple batching - added parameter "batch_size" to TopologyBuilder - added Batch class - adopted Kryo (de)serialization for Tuple and Batch - added Spout/Bolt output batchi

[jira] [Commented] (STORM-829) Hadoop dependency confusion

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-829?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705375#comment-14705375 ] ASF GitHub Bot commented on STORM-829: -- Github user YvonneIronberg commented on the pu

[GitHub] storm pull request: STORM-829: Hadoop dependency confusion

2015-08-20 Thread YvonneIronberg
Github user YvonneIronberg commented on the pull request: https://github.com/apache/storm/pull/690#issuecomment-133088728 Thanks for your review, @knusbaum ! I've followed your suggestions (title, rebase, remove hadoop-auth in storm-core/pom.xml). --- If your project is set

[jira] [Commented] (STORM-829) Hadoop dependency confusion

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-829?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705202#comment-14705202 ] ASF GitHub Bot commented on STORM-829: -- Github user knusbaum commented on the pull req

[GitHub] storm pull request: STORM-829

2015-08-20 Thread knusbaum
Github user knusbaum commented on the pull request: https://github.com/apache/storm/pull/690#issuecomment-133064686 Just a couple comments: - Would you please include the summary of the Jira in the title of the pull request - There are a bunch of commits here. Would you squas

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705190#comment-14705190 ] ASF GitHub Bot commented on STORM-978: -- Github user HeartSaVioR commented on the pull

[GitHub] storm pull request: STORM-978: Implemented EsLookupBolt

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/693#issuecomment-133063574 @alexpanov Could you describe more about ElasticsearchGetRequest and EsLookupResultOutput? Please refer https://github.com/apache/storm/pull/679/files -> READ

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705171#comment-14705171 ] ASF GitHub Bot commented on STORM-978: -- Github user alexpanov commented on the pull re

[GitHub] storm pull request: STORM-978: Implemented EsLookupBolt

2015-08-20 Thread alexpanov
Github user alexpanov commented on the pull request: https://github.com/apache/storm/pull/693#issuecomment-133060807 @HeartSaVioR Thanks. I have made the suggested changes and also added a README entry. --- If your project is set up for it, you can reply to this email and have yo

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705156#comment-14705156 ] ASF GitHub Bot commented on STORM-978: -- Github user HeartSaVioR commented on the pull

[GitHub] storm pull request: STORM-978: Implemented EsLookupBolt

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/693#issuecomment-133057909 @harshach Could you also take a look? --- 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

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705154#comment-14705154 ] ASF GitHub Bot commented on STORM-978: -- Github user HeartSaVioR commented on the pull

[GitHub] storm pull request: STORM-978: Implemented EsLookupBolt

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/693#issuecomment-133057299 @alexpanov Well done! Great improvements on tests, and great to add EsLookupBolt. Btw, could you please describe how to use EsLookupBolt to README.md?

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705140#comment-14705140 ] ASF GitHub Bot commented on STORM-978: -- Github user HeartSaVioR commented on a diff in

[GitHub] storm pull request: STORM-978: Implemented EsLookupBolt

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/693#discussion_r37544699 --- Diff: external/storm-elasticsearch/src/test/java/org/apache/storm/elasticsearch/bolt/TransportAddressesTest.java --- @@ -0,0 +1,64 @@ +package org

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705132#comment-14705132 ] ASF GitHub Bot commented on STORM-978: -- Github user HeartSaVioR commented on a diff in

[GitHub] storm pull request: STORM-978: Implemented EsLookupBolt

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/693#discussion_r37543881 --- Diff: external/storm-elasticsearch/src/test/java/org/apache/storm/elasticsearch/bolt/EsLookupBoltTest.java --- @@ -0,0 +1,102 @@ +package org.apac

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705101#comment-14705101 ] ASF GitHub Bot commented on STORM-978: -- Github user HeartSaVioR commented on a diff in

[GitHub] storm pull request: STORM-978: Implemented EsLookupBolt

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/693#discussion_r37541934 --- Diff: external/storm-elasticsearch/src/main/java/org/apache/storm/elasticsearch/bolt/EsLookupResultOutput.java --- @@ -0,0 +1,15 @@ +package org.a

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705087#comment-14705087 ] ASF GitHub Bot commented on STORM-978: -- Github user HeartSaVioR commented on a diff in

[GitHub] storm pull request: STORM-978: Implemented EsLookupBolt

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/693#discussion_r37540926 --- Diff: external/storm-elasticsearch/src/main/java/org/apache/storm/elasticsearch/ElasticsearchGetRequest.java --- @@ -0,0 +1,10 @@ +package org.apa

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14705083#comment-14705083 ] ASF GitHub Bot commented on STORM-978: -- Github user HeartSaVioR commented on a diff in

[GitHub] storm pull request: STORM-978: Implemented EsLookupBolt

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/693#discussion_r37540751 --- Diff: external/storm-elasticsearch/src/main/java/org/apache/storm/elasticsearch/ElasticsearchGetRequest.java --- @@ -0,0 +1,10 @@ +package org.apa

[jira] [Resolved] (STORM-949) On the topology summary UI page, last shown error should have the time and date

2015-08-20 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-949?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jungtaek Lim resolved STORM-949. Resolution: Fixed Fix Version/s: 0.11.0 Thanks [~jerrypeng], I merged this into master. > On

[GitHub] storm pull request: [STORM-949] On the topology summary UI page, a...

2015-08-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/675 --- 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 enabl

[jira] [Commented] (STORM-949) On the topology summary UI page, last shown error should have the time and date

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14704771#comment-14704771 ] ASF GitHub Bot commented on STORM-949: -- Github user asfgit closed the pull request at:

[jira] [Commented] (STORM-949) On the topology summary UI page, last shown error should have the time and date

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-949?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14704760#comment-14704760 ] ASF GitHub Bot commented on STORM-949: -- Github user HeartSaVioR commented on the pull

[GitHub] storm pull request: [STORM-949] On the topology summary UI page, a...

2015-08-20 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/675#issuecomment-132990727 Modified version of UI is what I and @knusbaum , @d2r suggested, and it seems that there're no other opinions now. Build failure is not related to. So I'm +1.

[jira] [Commented] (STORM-996) netty-unit-tests/test-batch demonstrates out-of-order delivery

2015-08-20 Thread Jungtaek Lim (JIRA)
[ https://issues.apache.org/jira/browse/STORM-996?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14704748#comment-14704748 ] Jungtaek Lim commented on STORM-996: I'm not sure it was same issue cause error message

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread Alex Panov (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14704648#comment-14704648 ] Alex Panov commented on STORM-978: -- [~kabhwan] please take a look at the pull request: ht

[GitHub] storm pull request: STORM-978: Implemented EsLookupBolt

2015-08-20 Thread alexpanov
GitHub user alexpanov opened a pull request: https://github.com/apache/storm/pull/693 STORM-978: Implemented EsLookupBolt https://issues.apache.org/jira/browse/STORM-978 You can merge this pull request into a Git repository by running: $ git pull https://github.com/alexpanov/st

[jira] [Commented] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14704647#comment-14704647 ] ASF GitHub Bot commented on STORM-978: -- GitHub user alexpanov opened a pull request:

[jira] [Assigned] (STORM-978) [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched documents from ES

2015-08-20 Thread Alex Panov (JIRA)
[ https://issues.apache.org/jira/browse/STORM-978?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alex Panov reassigned STORM-978: Assignee: Alex Panov > [storm-elasticsearch] Introduces Lookup(or Query)Bolt which emits matched >

[GitHub] storm pull request: STORM-997: Add proxy user functionality for st...

2015-08-20 Thread priyank5485
GitHub user priyank5485 opened a pull request: https://github.com/apache/storm/pull/692 STORM-997: Add proxy user functionality for storm hdfs connector. You can merge this pull request into a Git repository by running: $ git pull https://github.com/priyank5485/storm STORM-997

[jira] [Commented] (STORM-997) Add support for user specified UGI - (UserGroupInformation) for storm hdfs connector

2015-08-20 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/STORM-997?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14704422#comment-14704422 ] ASF GitHub Bot commented on STORM-997: -- GitHub user priyank5485 opened a pull request: