[GitHub] storm issue #1714: STORM-2125 Use Calcite's implementation of Rex Compiler

2016-10-06 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1714 Added full of scalar function tests which Calcite provides. Calcite 1.9.0 has some bugs so not all functions are working well, but it can be fixed from Calcite side. (I left comments to test

Are storm metrics reported through JMX too?

2016-10-06 Thread S G
Hi, We want to use Telegraf ( https://github.com/influxdata/telegraf/tree/master/plugins) for getting storm's metrics. But we do not want to add a HttpForwardingMetricsServer just to get the metrics and send them to telegraf. Other option is to use Jolokia (https://jolokia.org/) that can read

[GitHub] storm issue #1714: STORM-2125 Use Calcite's implementation of Rex Compiler

2016-10-06 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1714 Addressed documenting covered feature from other PR: #1725 --- 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

[GitHub] storm pull request #1725: STORM-2135 document covered features of SQL refere...

2016-10-06 Thread HeartSaVioR
GitHub user HeartSaVioR opened a pull request: https://github.com/apache/storm/pull/1725 STORM-2135 document covered features of SQL reference in Calcite * NOTE: This should be merged after STORM-2125 is merged to master * add a new page "storm-sql-reference" based on

Re: [DISCUSS] Plan for merge SQE and Storm SQL

2016-10-06 Thread Jungtaek Lim
For preventing this discussion to be staled, I'd like to put a note how Storm SQL is going now. (including the change after my tech. analysis) 1. There're 6 pull requests opened regarding Storm SQL. https://github.com/apache/storm/pulls/HeartSaVioR 2. When STORM-2125

[GitHub] storm issue #1724: STORM-2131: Add blob command to worker-launcher, make sto...

2016-10-06 Thread knusbaum
Github user knusbaum commented on the issue: https://github.com/apache/storm/pull/1724 @kishorvpatil @revans2 @jerrypeng --- 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 issue #1724: STORM-2131: Add blob command to worker-launcher, make sto...

2016-10-06 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/storm/pull/1724 +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