[GitHub] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-04-02 Thread mjsax
Github user mjsax commented on the issue:

https://github.com/apache/flink/pull/1157
  
@HuangWHWHW Can you please close this PR? @RalphSu is working on this now 
(cf #3615 ). Thanks.


---
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] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-03-26 Thread RalphSu
Github user RalphSu commented on the issue:

https://github.com/apache/flink/pull/1157
  
I added https://github.com/apache/flink/pull/3615, welcome for comments.


---
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] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-03-06 Thread mjsax
Github user mjsax commented on the issue:

https://github.com/apache/flink/pull/1157
  
@RalphSu We should wait for @HuangWHWHW to reply... If no response comes 
within 3 days you can take over. Also, the JIRA should get assigned to you for 
this case: https://issues.apache.org/jira/browse/FLINK-2720


---
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] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-03-05 Thread RalphSu
Github user RalphSu commented on the issue:

https://github.com/apache/flink/pull/1157
  
okey, would try file a PR on this soon


---
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] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-03-05 Thread mjsax
Github user mjsax commented on the issue:

https://github.com/apache/flink/pull/1157
  
@HuangWHWHW Are you still working on this? @RalphSu Are you interested in 
talking this over?


---
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] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-03-05 Thread RalphSu
Github user RalphSu commented on the issue:

https://github.com/apache/flink/pull/1157
  
?


---
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] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-03-05 Thread RalphSu
Github user RalphSu commented on the issue:

https://github.com/apache/flink/pull/1157
  
Here it looks user need to change code to make metrics work


---
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] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-03-05 Thread RalphSu
Github user RalphSu commented on the issue:

https://github.com/apache/flink/pull/1157
  
Here it looks user need to change code to make metrics work


---
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] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-03-05 Thread RalphSu
Github user RalphSu commented on the issue:

https://github.com/apache/flink/pull/1157
  
Any progress on this one? Looks this has been stop for a while.


---
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] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-01-13 Thread mjsax
Github user mjsax commented on the issue:

https://github.com/apache/flink/pull/1157
  
Did not have a chance to work on this either -- I am in a new job, too :)

You can pick this up again if you are still interested.


---
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] flink issue #1157: [FLINK-2720][storm-compatibility]Add Storm-CountMetric fo...

2017-01-13 Thread HuangWHWHW
Github user HuangWHWHW commented on the issue:

https://github.com/apache/flink/pull/1157
  
@mjsax Hi, very sorry to hang you out. I just did some other jobs last year 
 because of some special circumstances.
How is the progress been recently?
Please let me know if you need some help here.
Very sorry before.


---
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.
---