[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-21 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r93530698 --- Diff: storm-dist/binary/final-package/src/main/assembly/binary.xml --- @@ -446,6 +446,29 @@ storm*jar

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-20 Thread ptgoetz
Github user ptgoetz commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r93312373 --- Diff: storm-dist/binary/final-package/src/main/assembly/binary.xml --- @@ -446,6 +446,29 @@ storm*jar

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-15 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92736752 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/PMMLPredictorBolt.java --- @@ -0,0 +1,118 @@ +/* + * Licensed to the Apache Software

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-15 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92736718 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/PMMLPredictorBolt.java --- @@ -0,0 +1,118 @@ +/* + * Licensed to the Apache Software

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-15 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92736561 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/PMMLPredictorBolt.java --- @@ -0,0 +1,118 @@ +/* + * Licensed to the Apache Software

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-15 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92736547 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/spout/test/KafkaSpoutTopologyMainNamedTopics.java --- @@ -39,7 +39,6 @@ import

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-15 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92736541 --- Diff: examples/storm-pmml-examples/src/main/java/org/apache/storm/pmml/JpmmlRunnerTestTopology.java --- @@ -0,0 +1,167 @@ +/* + * Licensed to the

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-15 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92736555 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/PMMLPredictorBolt.java --- @@ -0,0 +1,118 @@ +/* + * Licensed to the Apache Software

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-15 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92736543 --- Diff: examples/storm-pmml-examples/src/main/java/org/apache/storm/pmml/RawInputFromCSVSpout.java --- @@ -0,0 +1,133 @@ +/* + * Licensed to the

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-15 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92736533 --- Diff: examples/storm-pmml-examples/src/main/java/org/apache/storm/pmml/JpmmlRunnerTestTopology.java --- @@ -0,0 +1,167 @@ +/* + * Licensed to the

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-15 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92736485 --- Diff: examples/storm-pmml-examples/src/main/resources/KNIME_PMML_4.1_Examples_single_audit_logreg.xml --- @@ -0,0 +1,259 @@ + --- End diff --

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-15 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92736455 --- Diff: external/storm-pmml/README.md --- @@ -0,0 +1,104 @@ +#Storm PMML Bolt + Storm integration to load PMML models and compute predictive scores for

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92328626 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/PMMLPredictorBolt.java --- @@ -0,0 +1,118 @@ +/* + * Licensed to the Apache

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92328164 --- Diff: external/storm-kafka-client/src/test/java/org/apache/storm/kafka/spout/test/KafkaSpoutTopologyMainNamedTopics.java --- @@ -39,7 +39,6 @@

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92330433 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/PMMLPredictorBolt.java --- @@ -0,0 +1,118 @@ +/* + * Licensed to the Apache

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92327978 --- Diff: examples/storm-pmml-examples/src/main/java/org/apache/storm/pmml/RawInputFromCSVSpout.java --- @@ -0,0 +1,133 @@ +/* + * Licensed to

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92328530 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/PMMLPredictorBolt.java --- @@ -0,0 +1,118 @@ +/* + * Licensed to the Apache

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92330245 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/model/Stream.java --- @@ -0,0 +1,74 @@ +/* + * Licensed to the Apache

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92331912 --- Diff: examples/storm-pmml-examples/src/main/java/org/apache/storm/pmml/JpmmlRunnerTestTopology.java --- @@ -0,0 +1,167 @@ +/* + * Licensed

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92331432 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/runner/jpmml/JPmmlModelRunner.java --- @@ -0,0 +1,134 @@ +/* + * Licensed to

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92329373 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/PMMLPredictorBolt.java --- @@ -0,0 +1,118 @@ +/* + * Licensed to the Apache

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92326957 --- Diff: examples/storm-pmml-examples/src/main/java/org/apache/storm/pmml/JpmmlRunnerTestTopology.java --- @@ -0,0 +1,167 @@ +/* + * Licensed

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92328927 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/PMMLPredictorBolt.java --- @@ -0,0 +1,118 @@ +/* + * Licensed to the Apache

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92328525 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/PMMLPredictorBolt.java --- @@ -0,0 +1,118 @@ +/* + * Licensed to the Apache

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread ptgoetz
Github user ptgoetz commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92257368 --- Diff: examples/storm-pmml-examples/src/main/resources/KNIME_PMML_4.1_Examples_single_audit_logreg.xml --- @@ -0,0 +1,259 @@ + --- End diff --

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92249343 --- Diff: external/storm-pmml/README.md --- @@ -0,0 +1,104 @@ +#Storm PMML Bolt + Storm integration to load PMML models and compute predictive scores for

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92247952 --- Diff: examples/storm-pmml-examples/src/main/resources/KNIME_PMML_4.1_Examples_single_audit_logreg.xml --- @@ -0,0 +1,259 @@ + --- End diff --

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92246988 --- Diff: external/storm-pmml/README.md --- @@ -0,0 +1,104 @@ +#Storm PMML Bolt + Storm integration to load PMML models and compute predictive scores for

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread ptgoetz
Github user ptgoetz commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92244506 --- Diff: examples/storm-pmml-examples/src/main/resources/Audit.50.csv --- @@ -0,0 +1,51 @@

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-13 Thread ptgoetz
Github user ptgoetz commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r92243287 --- Diff: external/storm-pmml/README.md --- @@ -0,0 +1,104 @@ +#Storm PMML Bolt + Storm integration to load PMML models and compute predictive scores

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-11 Thread hmcl
Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r91879180 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/runner/jpmml/JPmmlModelRunner.java --- @@ -0,0 +1,119 @@ +/* + * Licensed to the

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-08 Thread csivaguru
Github user csivaguru commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r91615776 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/runner/jpmml/JpmmlFactory.java --- @@ -0,0 +1,88 @@ +/* + * Licensed to the

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-08 Thread ptgoetz
Github user ptgoetz commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r91586225 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/runner/jpmml/JpmmlFactory.java --- @@ -0,0 +1,88 @@ +/* + * Licensed to the Apache

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-07 Thread csivaguru
Github user csivaguru commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r91385214 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/runner/jpmml/JpmmlFactory.java --- @@ -0,0 +1,88 @@ +/* + * Licensed to the

[GitHub] storm pull request #1816: STORM-2223: PMMLBolt

2016-12-07 Thread vesense
Github user vesense commented on a diff in the pull request: https://github.com/apache/storm/pull/1816#discussion_r91297902 --- Diff: external/storm-pmml/src/main/java/org/apache/storm/pmml/runner/ModelRunner.java --- @@ -0,0 +1,30 @@ +/* + * Licensed to the Apache