[jira] [Commented] (PIO-18) Documentation for setting up the project for developers

2016-08-06 Thread Hyukjin Kwon (JIRA)
[ https://issues.apache.org/jira/browse/PIO-18?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15410824#comment-15410824 ] Hyukjin Kwon commented on PIO-18: - I am using IntelliJ. I wanted to go ahead with this but I f

[jira] [Created] (PIO-18) Documentation for setting up the project for developers

2016-08-06 Thread Hyukjin Kwon (JIRA)
Hyukjin Kwon created PIO-18: --- Summary: Documentation for setting up the project for developers Key: PIO-18 URL: https://issues.apache.org/jira/browse/PIO-18 Project: PredictionIO Issue Type: Improv

[GitHub] incubator-predictionio issue #269: ActionML Master merge PR, discussion only

2016-08-06 Thread pferrel
Github user pferrel commented on the issue: https://github.com/apache/incubator-predictionio/pull/269 I think this is ready for review. The main things here are changes to support trimming data in the PEvents and LEvents somewhat like the formerly experimental Cleanup App did

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793297 --- Diff: tools/build.sbt --- @@ -23,19 +21,22 @@ libraryDependencies ++= Seq( "io.spray" %% "spray-can" % "1.3.3",

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793267 --- Diff: examples/scala-parallel-similarproduct/filterbyyear/src/main/scala/ALSAlgorithm.scala --- @@ -45,7 +45,7 @@ class ALSAlgorithm(val

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793249 --- Diff: examples/scala-parallel-similarproduct/filterbyyear/build.sbt --- @@ -7,6 +7,6 @@ name := "template-scala-parallel-similarproduct"

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793283 --- Diff: examples/scala-parallel-similarproduct/filterbyyear/src/main/scala/DataSource.scala --- @@ -13,16 +16,21 @@ import org.apache.spar

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793260 --- Diff: examples/scala-parallel-similarproduct/filterbyyear/engine.json --- @@ -5,6 +5,10 @@ "datasource": { "params" : {

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793246 --- Diff: docs/manual/data/versions.yml --- @@ -1,6 +1,6 @@ pio: 0.9.6 spark: 1.5.2 spark_download_filename: spark-1.5.1-bin-ha

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793235 --- Diff: data/src/main/scala/io/prediction/data/store/PEventStore.scala --- @@ -24,6 +24,8 @@ import org.joda.time.DateTime import org.a

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793218 --- Diff: data/src/main/scala/io/prediction/data/storage/hbase/HBPEvents.scala --- @@ -109,4 +109,25 @@ class HBPEvents(client: HBClient, con

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793228 --- Diff: data/src/main/scala/io/prediction/data/storage/jdbc/JDBCPEvents.scala --- @@ -155,6 +156,22 @@ class JDBCPEvents(client: String, co

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793210 --- Diff: data/src/main/scala/io/prediction/data/storage/PEvents.scala --- @@ -179,4 +179,8 @@ trait PEvents extends Serializable { *

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793183 --- Diff: data/build.sbt --- @@ -44,9 +44,9 @@ libraryDependencies ++= Seq( "org.json4s" %% "json4s-native" % json4sVersio

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793199 --- Diff: data/src/main/scala/io/prediction/data/storage/Models.scala --- @@ -78,3 +78,6 @@ class ModelSerializer extends CustomSerializer[Mod

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793171 --- Diff: core/src/test/scala/io/prediction/core/SelfCleaningDataSourceTest.scala --- @@ -0,0 +1,78 @@ +package io.prediction.core.test

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793158 --- Diff: core/src/main/scala/io/prediction/core/SelfCleaningDataSource.scala --- @@ -0,0 +1,313 @@ +package io.prediction.core +

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793137 --- Diff: core/build.sbt --- @@ -27,13 +27,14 @@ libraryDependencies ++= Seq( "io.spray" %% "spray-routing"% "1.3.3",

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793055 --- Diff: bin/pio-stop-all --- @@ -43,3 +43,18 @@ if [ -e ${PIDFILE} ]; then cat ${PIDFILE} | xargs kill rm ${PIDFILE} fi

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793121 --- Diff: build.sbt --- @@ -193,3 +193,8 @@ concurrentRestrictions in Global := Seq( Tags.limitAll( 1 ) ) +parallelExecu

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793051 --- Diff: bin/pio-start-all --- @@ -57,6 +57,26 @@ else exit 1 fi +#PGSQL +pgsqlStatus="$(ps auxwww | grep postgres

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793112 --- Diff: build.sbt --- @@ -18,9 +18,9 @@ import UnidocKeys._ name := "pio" -version in ThisBuild := "0.9.6" +version

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793029 --- Diff: RELEASE.md --- @@ -65,7 +106,9 @@ March 4th, 2015 | [Release Notes](https://predictionio.atlassian.net/jira/secure ###v0

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793019 --- Diff: README.md --- @@ -76,21 +71,17 @@ Status](https://gemnasium.com/PredictionIO/PredictionIO.svg)](https://gemnasium. Keep

[GitHub] incubator-predictionio pull request #269: ActionML Master merge PR, discussi...

2016-08-06 Thread pferrel
Github user pferrel commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/269#discussion_r73793046 --- Diff: bin/install.sh --- @@ -389,16 +409,10 @@ case $source_setup in ${SED_CMD} "s|# PIO_STORAGE_SOURCES_LOCALFS|PIO_STORAGE_SOU

[GitHub] incubator-predictionio issue #269: Master

2016-08-06 Thread pferrel
Github user pferrel commented on the issue: https://github.com/apache/incubator-predictionio/pull/269 Just to track progress while merging the ActionML fork. --- 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

[GitHub] incubator-predictionio pull request #269: Master

2016-08-06 Thread pferrel
GitHub user pferrel opened a pull request: https://github.com/apache/incubator-predictionio/pull/269 Master You can merge this pull request into a Git repository by running: $ git pull https://github.com/pferrel/PredictionIO master Alternatively you can review and apply these

[GitHub] incubator-predictionio pull request #268: changed mistyped admin-server-port...

2016-08-06 Thread maniteja6799
GitHub user maniteja6799 opened a pull request: https://github.com/apache/incubator-predictionio/pull/268 changed mistyped admin-server-port to admin-server-ip the option for admin server ip in pio deploy command is written as admin-server-port, changed that to admin-server-ip. You

[jira] [Resolved] (PIO-15) Create KEYS file with public keys of Release Managers for PredictionIO

2016-08-06 Thread Suneel Marthi (JIRA)
[ https://issues.apache.org/jira/browse/PIO-15?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Suneel Marthi resolved PIO-15. -- Resolution: Implemented > Create KEYS file with public keys of Release Managers for PredictionIO >