[GitHub] storm pull request: add dotNet adapter.

2015-07-02 Thread ziyunhx
GitHub user ziyunhx opened a pull request:

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

add dotNet adapter.



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

$ git pull https://github.com/ziyunhx/storm master

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

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


commit 8538ad35e1cb20f4e14eef968bab05a969a74fe9
Author: Carey Tzou ziyu...@gmail.com
Date:   2015-07-02T08:12:05Z

add dotNet adapter.




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[jira] [Created] (STORM-924) Set the file mode of the files included when packaging release packages

2015-07-02 Thread caofangkun (JIRA)
caofangkun created STORM-924:


 Summary: Set the file mode of the files included when packaging 
release packages
 Key: STORM-924
 URL: https://issues.apache.org/jira/browse/STORM-924
 Project: Apache Storm
  Issue Type: Improvement
Affects Versions: 0.11.0
Reporter: caofangkun
Assignee: caofangkun
Priority: Minor






--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (STORM-924) Set the file mode of the files included when packaging release packages

2015-07-02 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/STORM-924?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14611678#comment-14611678
 ] 

ASF GitHub Bot commented on STORM-924:
--

GitHub user caofangkun opened a pull request:

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

STORM-924:Set the file mode of the files included when packaging release 
packages



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

$ git pull https://github.com/caofangkun/apache-storm storm-924

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

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


commit 5d26820167e83358efbb4c05a03976d9c7ea
Author: caofangkun caofang...@gmail.com
Date:   2015-05-25T03:27:01Z

Merge pull request #22 from apache/master

Merge from apache/storm to caofangkun/apache-storm

commit 5122dea1216c9b6125e8fd5364f90236c84ae196
Author: caofangkun caofang...@gmail.com
Date:   2015-06-01T06:30:22Z

Merge pull request #23 from apache/master

Merge from apache/storm to caofangkun/apache-storm

commit 0f71a7db32b47208ff7373e8b1c75f9f5ebaf9b4
Author: caofangkun caofang...@gmail.com
Date:   2015-06-12T03:34:44Z

Merge pull request #24 from apache/master

 Merge from apache/storm to caofangkun/apache-storm

commit 03d276069797cc993c7078f7ece513802960d40d
Author: caofangkun caofang...@gmail.com
Date:   2015-06-17T02:22:13Z

Merge pull request #25 from apache/master

Merge from apache/storm to caofangkun/apache-storm

commit 469b7f56f910bc9f9b8d3af7670f86fbdb5c53ed
Author: caofangkun caofang...@gmail.com
Date:   2015-06-23T03:22:20Z

Merge pull request #26 from apache/master

Merge from apache/storm to caofangkun/apache-storm

commit dddaacbf51d37f97e072df0eed4d14c09cdda453
Author: caofangkun caofang...@gmail.com
Date:   2015-06-26T02:28:32Z

Merge pull request #27 from apache/master

Merge from apache/storm to caofangkun/apache-storm

commit 874f5cf3312db33c2658c023f702c7f62916fbec
Author: caofangkun caofang...@gmail.com
Date:   2015-07-02T08:53:57Z

STORM-924:Set the file mode of the files included when packaging the 
release packages




 Set the file mode of the files included when packaging release packages
 ---

 Key: STORM-924
 URL: https://issues.apache.org/jira/browse/STORM-924
 Project: Apache Storm
  Issue Type: Improvement
Affects Versions: 0.11.0
Reporter: caofangkun
Assignee: caofangkun
Priority: Minor





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] storm pull request: add dotNet adapter.

2015-07-02 Thread ziyunhx
Github user ziyunhx commented on the pull request:

https://github.com/apache/storm/pull/612#issuecomment-117957706
  
the StormSimple.exe source is at 
https://github.com/ziyunhx/storm-net-adapter/tree/master/StormSimple. I can't 
find where it should be.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] storm pull request: STORM-924:Set the file mode of the files inclu...

2015-07-02 Thread lazyval
Github user lazyval commented on the pull request:

https://github.com/apache/storm/pull/613#issuecomment-118044177
  
:+1: 

For those who wonders what this octal numbers mean, there is a great 
[*humanizator*](http://permissions-calculator.org/decode/0644/).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] storm pull request: add dotNet adapter.

2015-07-02 Thread lazyval
Github user lazyval commented on the pull request:

https://github.com/apache/storm/pull/612#issuecomment-118042465
  
Code aside, I guess storing *compiled* binaries and libraries in git isn't 
quite a good idea. 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


Tick Tuple

2015-07-02 Thread Pradheep s
HI,

I have a topology where there are 10 instances of a spout connected to a
bolt.
I have set the tick tuple frequency of 1 sec.
When i run the topology, i see that before the tick tuple starts, storm
calls nexttuple function in all the spouts and the tuples are emitted.
So i get a large number of tuples in the first second say 100,000
once the tick tuple starts , the next second i receive around 8000 tuples
and averages around after that..
So my question is why storm calls nextuple in the spouts before starting
the tick tuple?Is this a bug in storm?Can this be fixed?

Thanks,
Pradheep


[jira] [Commented] (STORM-919) Gathering worker and supervisor process information (CPU/Memory)

2015-07-02 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/STORM-919?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14612016#comment-14612016
 ] 

ASF GitHub Bot commented on STORM-919:
--

Github user lazyval commented on the pull request:

https://github.com/apache/storm/pull/608#issuecomment-118050871
  
It might be too late for this particular PR, but it makes sense to separate 
human changes and generated code into two separate commits. Otherwise it's 
really hard to review the code. 


 Gathering worker and supervisor process information (CPU/Memory)
 

 Key: STORM-919
 URL: https://issues.apache.org/jira/browse/STORM-919
 Project: Apache Storm
  Issue Type: New Feature
Reporter: Shyam Rajendran
Assignee: Shyam Rajendran
Priority: Minor

 It would be useful to have supervisor and worker process related information 
 such as %cpu utilization, JVM memory and network bandwidth available to 
 NIMBUS which would be useful for resource aware scheduler implementation 
 later on. As a beginning, the information can be piggybacked on the existing 
 heartbeats into the ZK or to the pacemaker as required. 
 Related JIRAs
 STORM-177
 STORM-891
 STORM-899



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] storm pull request: STORM-919 Gathering worker and supervisor proc...

2015-07-02 Thread lazyval
Github user lazyval commented on the pull request:

https://github.com/apache/storm/pull/608#issuecomment-118050871
  
It might be too late for this particular PR, but it makes sense to separate 
human changes and generated code into two separate commits. Otherwise it's 
really hard to review the code. 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] storm pull request: add dotNet adapter.

2015-07-02 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request:

https://github.com/apache/storm/pull/612#issuecomment-118058205
  
@ziyunhx

Agreed to @lazyval. 

- Project setting files should be removed from repo, which includes PDB, 
etc. (Maybe csproj too?)
- If we can make DLLs from source code, DLLs should be removed from repo.

Btw, I wonder Storm community can maintain non-JVM .net codes (which is 
actually a project) within JVM project. It also adds/changes 1+ lines to 
storm-core, which is big compared to origin storm-core.

Please note that multilang implementations in storm-core are simple enough 
to represent just one file.
They don't require additional libraries installed, too.

There're some advanced Storm multilang projects, for example, 
StreamParse(https://github.com/Parsely/streamparse). 
.Net multilang adapter could be better to be maintained like StreamParse, I 
mean, outer Storm project.
It seems to be more natural.

How about posting new feature to user / dev mailing list and gather a 
consensus that we would be better to include this?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] storm pull request: add dotNet adapter.

2015-07-02 Thread ziyunhx
Github user ziyunhx closed the pull request at:

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


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] storm pull request: STORM-924:Set the file mode of the files inclu...

2015-07-02 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request:

https://github.com/apache/storm/pull/613#issuecomment-118211480
  
I like the change. It would be better to set files' mode explicitly 
whenever it needs.

Only thing that I'm afraid of is unrelated commits, which are having no 
meaning for Apache Storm repo.
Could you handle it?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---