[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-12-02 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-161362165 maybe need a modify list to display JStorm's update --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-12-02 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-161361062 @wuchong Good to hear. Visually they look nearly identical, and the license looks compatible. --- If your project is set up for it, you can reply to this email and have

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-12-02 Thread wuchong
Github user wuchong commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-161286041 Hi @ptgoetz , thanks for the note. We have tried a lot of alternative solutions, such as: d3, chart.js, xCharts, chartist.js and ECharts. Finally we find that [ECharts](h

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-12-01 Thread bastiliu
Github user bastiliu commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-161157842 @ptgoetz Thanks for the information. We will find a new solution which is compatible with ASF license policy. --- If your project is set up for it, you can reply to thi

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-12-01 Thread bastiliu
Github user bastiliu commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-161157816 @ptgoetz Thanks for the information. We find a new solution which is compatible with ASF license policy. --- If your project is set up for it, you can reply to this ema

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-12-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/904 --- 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

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-12-01 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-16124 Thanks @bastiliu I merged this into the jstorm-import. **One important thing to note:** This pull request introduced highchart.js which has a commercial l

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-11-25 Thread zhuoliu
Github user zhuoliu commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-159692305 +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

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-11-25 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-159662148 +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

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-11-25 Thread darionyaphet
Github user darionyaphet commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-159637347 +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

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-11-25 Thread wuchong
Github user wuchong commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-159601051 +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

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-11-25 Thread hustfxj
Github user hustfxj commented on the pull request: https://github.com/apache/storm/pull/904#issuecomment-159601049 +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

[GitHub] storm pull request: STORM-1353 Update "jstorm-import" branch to th...

2015-11-25 Thread bastiliu
GitHub user bastiliu opened a pull request: https://github.com/apache/storm/pull/904 STORM-1353 Update "jstorm-import" branch to the latest JStorm release 2.1.0 Update the reference banch "jstorm-import" with latest JStorm release 2.1.0, since many JStorm features listed in JStorm m