[jira] [Resolved] (PIO-53) Create per-commit Docker image for testing

2017-02-26 Thread Donald Szeto (JIRA)

 [ 
https://issues.apache.org/jira/browse/PIO-53?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Donald Szeto resolved PIO-53.
-
Resolution: Fixed

> Create per-commit Docker image for testing
> --
>
> Key: PIO-53
> URL: https://issues.apache.org/jira/browse/PIO-53
> Project: PredictionIO
>  Issue Type: Test
>Reporter: Donald Szeto
>Assignee: Donald Szeto
> Fix For: 0.11.0
>
>




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (PIO-45) SelfCleaningDatasource erases all data

2017-02-26 Thread Pat Ferrel (JIRA)

[ 
https://issues.apache.org/jira/browse/PIO-45?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885085#comment-15885085
 ] 

Pat Ferrel commented on PIO-45:
---

[~emergentorder] any movement on this? We want to release this week. I'll test 
if that helps.

> SelfCleaningDatasource erases all data
> --
>
> Key: PIO-45
> URL: https://issues.apache.org/jira/browse/PIO-45
> Project: PredictionIO
>  Issue Type: Bug
>Affects Versions: 0.10.0-incubating
>Reporter: Pat Ferrel
>Assignee: Alexander  Merritt
>Priority: Blocker
> Fix For: 0.11.0
>
> Attachments: import_handmade_simple.py, 
> sample-time-window-and-downsample-data.txt
>
>
> as integrated into the UR, in the integration-test, the SelfCleaningDataset 
> erases all data. This feature works fine in the AML version of PIO.
> Although not tested one could assume that this would be true with any other 
> Datasource in other templates.
> [~emergentorder] can you check to see if the PIO merge was done correctly.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] incubator-predictionio issue #351: Low to no activity

2017-02-26 Thread lucasbm88
Github user lucasbm88 commented on the issue:

https://github.com/apache/incubator-predictionio/issues/351
  
I do not see this way, perhaps you did not subscribe to the project 
developers mailing list: dev@predictionio.incubator.apache.org


---
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] incubator-predictionio issue #336: [PIO-49] Add support for Elasticsearch 5....

2017-02-26 Thread marevol
Github user marevol commented on the issue:

https://github.com/apache/incubator-predictionio/pull/336
  
Right. To use ES5, please replace pio-data-elasticsearch1-assembly-\*.jar 
with pio-data-elasticsearch-assembly-\*.jar in lib/spark directory.

To change JARs location(ex. lib/spark), we can modify JAR lookup handing in 
compute-classpath.sh and Common.scala for #352.

> The default for 0.11 is ES1, with a param for ES5.

ES 1.x was EOLed.
https://www.elastic.co/jp/support/eol




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


[jira] [Commented] (PIO-49) Add support for Elasticsearch 5.x

2017-02-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/PIO-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885030#comment-15885030
 ] 

ASF GitHub Bot commented on PIO-49:
---

Github user marevol commented on the issue:

https://github.com/apache/incubator-predictionio/pull/336
  
Right. To use ES5, please replace pio-data-elasticsearch1-assembly-\*.jar 
with pio-data-elasticsearch-assembly-\*.jar in lib/spark directory.

To change JARs location(ex. lib/spark), we can modify JAR lookup handing in 
compute-classpath.sh and Common.scala for #352.

> The default for 0.11 is ES1, with a param for ES5.

ES 1.x was EOLed.
https://www.elastic.co/jp/support/eol




> Add support for Elasticsearch 5.x
> -
>
> Key: PIO-49
> URL: https://issues.apache.org/jira/browse/PIO-49
> Project: PredictionIO
>  Issue Type: Improvement
>Reporter: Shinsuke Sugaya
>
> We work on meta/event storage support for Elasticsearch 5.x.
> Although Elasticsearch 2.x does not allow dots in field names,
> Elasticsearch 5.x supports it. So, it's better to upgrade to ES 5.x release.
> Since ES 5.x provides Java Rest API client, we replaced
> Transport communication with HTTP one. Therefore, our fix
> uses HTTP(9200 port) only.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] incubator-predictionio issue #336: [PIO-49] Add support for Elasticsearch 5....

2017-02-26 Thread pferrel
Github user pferrel commented on the issue:

https://github.com/apache/incubator-predictionio/pull/336
  
@dszeto I meant `lib` = unmanaged, mistyped. 

This still sounds good. 

What will populate `lib/spark`? Maybe a flag in some sbt file or can a 
param be passed through `pio build`? If the later we have the equivalent of a 
maven build profile, sorta ad hoc but all good. The default for 0.11 is ES1, 
with a param for ES5. 

@marevol this means the params in pio-env.sh do not need to flag ES5, there 
should be only ELASTICSEARCH and it applies to whichever is in `lib/spark` and 
installed right?


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


[jira] [Commented] (PIO-49) Add support for Elasticsearch 5.x

2017-02-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/PIO-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15885011#comment-15885011
 ] 

ASF GitHub Bot commented on PIO-49:
---

Github user pferrel commented on the issue:

https://github.com/apache/incubator-predictionio/pull/336
  
@dszeto I meant `lib` = unmanaged, mistyped. 

This still sounds good. 

What will populate `lib/spark`? Maybe a flag in some sbt file or can a 
param be passed through `pio build`? If the later we have the equivalent of a 
maven build profile, sorta ad hoc but all good. The default for 0.11 is ES1, 
with a param for ES5. 

@marevol this means the params in pio-env.sh do not need to flag ES5, there 
should be only ELASTICSEARCH and it applies to whichever is in `lib/spark` and 
installed right?


> Add support for Elasticsearch 5.x
> -
>
> Key: PIO-49
> URL: https://issues.apache.org/jira/browse/PIO-49
> Project: PredictionIO
>  Issue Type: Improvement
>Reporter: Shinsuke Sugaya
>
> We work on meta/event storage support for Elasticsearch 5.x.
> Although Elasticsearch 2.x does not allow dots in field names,
> Elasticsearch 5.x supports it. So, it's better to upgrade to ES 5.x release.
> Since ES 5.x provides Java Rest API client, we replaced
> Transport communication with HTTP one. Therefore, our fix
> uses HTTP(9200 port) only.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] incubator-predictionio issue #352: [PIO-49] Add support for Elasticsearch 5 ...

2017-02-26 Thread dszeto
Github user dszeto commented on the issue:

https://github.com/apache/incubator-predictionio/pull/352
  
Hi @marevol , I have finished working on PIO-53, which allows unit and 
integration tests be updated per commit and removed reliance on a cached Docker 
image on Docker Hub. You might want to check that out while I review the rest 
of the code. 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] incubator-predictionio issue #352: [PIO-49] Add support for Elasticsearch 5 ...

2017-02-26 Thread dszeto
Github user dszeto commented on the issue:

https://github.com/apache/incubator-predictionio/pull/352
  
Adding @haginot to this thread.


---
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] incubator-predictionio issue #353: Update link to working template of OpenNL...

2017-02-26 Thread dszeto
Github user dszeto commented on the issue:

https://github.com/apache/incubator-predictionio/pull/353
  
@vshwnth2 Are you still maintaining your template? Are you okay with this 
change? Will merge this if there isn't a reply in a couple days.


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


[jira] [Commented] (PIO-49) Add support for Elasticsearch 5.x

2017-02-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/PIO-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15884989#comment-15884989
 ] 

ASF GitHub Bot commented on PIO-49:
---

Github user dszeto commented on the issue:

https://github.com/apache/incubator-predictionio/pull/352
  
Adding @haginot to this thread.


> Add support for Elasticsearch 5.x
> -
>
> Key: PIO-49
> URL: https://issues.apache.org/jira/browse/PIO-49
> Project: PredictionIO
>  Issue Type: Improvement
>Reporter: Shinsuke Sugaya
>
> We work on meta/event storage support for Elasticsearch 5.x.
> Although Elasticsearch 2.x does not allow dots in field names,
> Elasticsearch 5.x supports it. So, it's better to upgrade to ES 5.x release.
> Since ES 5.x provides Java Rest API client, we replaced
> Transport communication with HTTP one. Therefore, our fix
> uses HTTP(9200 port) only.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (PIO-49) Add support for Elasticsearch 5.x

2017-02-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/PIO-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15884988#comment-15884988
 ] 

ASF GitHub Bot commented on PIO-49:
---

Github user dszeto commented on the issue:

https://github.com/apache/incubator-predictionio/pull/352
  
Hi @marevol , I have finished working on PIO-53, which allows unit and 
integration tests be updated per commit and removed reliance on a cached Docker 
image on Docker Hub. You might want to check that out while I review the rest 
of the code. Thanks!


> Add support for Elasticsearch 5.x
> -
>
> Key: PIO-49
> URL: https://issues.apache.org/jira/browse/PIO-49
> Project: PredictionIO
>  Issue Type: Improvement
>Reporter: Shinsuke Sugaya
>
> We work on meta/event storage support for Elasticsearch 5.x.
> Although Elasticsearch 2.x does not allow dots in field names,
> Elasticsearch 5.x supports it. So, it's better to upgrade to ES 5.x release.
> Since ES 5.x provides Java Rest API client, we replaced
> Transport communication with HTTP one. Therefore, our fix
> uses HTTP(9200 port) only.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] incubator-predictionio issue #336: [PIO-49] Add support for Elasticsearch 5....

2017-02-26 Thread marevol
Github user marevol commented on the issue:

https://github.com/apache/incubator-predictionio/pull/336
  
Please see https://github.com/apache/incubator-predictionio/pull/352

- Move storage implementation to storage directory
- Put storage assembly JARs into lib/spark in distribution ZIP file
- To check ES1 support, default ES JAR in ZIP is for ES1 (please replace 
with ES5 before release)

If you have better location(lib/spark and lib/extra), please change them.


---
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] incubator-predictionio pull request #353: Update link to working template of...

2017-02-26 Thread infoquestsolutions
GitHub user infoquestsolutions opened a pull request:

https://github.com/apache/incubator-predictionio/pull/353

Update link to working template of OpenNLP-SentimentAnalysis-Template

Updated template OpenNLP-SentimentAnalysis-Template to be compatible with 
Apache PIO.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/infoquestsolutions/incubator-predictionio 
patch-1

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-predictionio/pull/353.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #353


commit 5b224f7f74e60f948d48b2604096aecb9723cf7e
Author: infoquestsolutions 
Date:   2017-02-26T15:36:00Z

Update link to working template of OpenNLP-SentimentAnalysis-Template

Updated template OpenNLP-SentimentAnalysis-Template to be compatible with 
Apache PIO.




---
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] incubator-predictionio pull request #349: [PIO-54] Avoid access key started ...

2017-02-26 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/incubator-predictionio/pull/349


---
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] incubator-predictionio pull request #352: [PIO-49] Add support for Elasticse...

2017-02-26 Thread marevol
GitHub user marevol opened a pull request:

https://github.com/apache/incubator-predictionio/pull/352

[PIO-49] Add support for Elasticsearch 5 and move storage implementation



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/marevol/incubator-predictionio move-storages

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-predictionio/pull/352.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #352


commit 3227fc7c270729df5e45fd069f93201851dd
Author: Shinsuke Sugaya 
Date:   2017-02-26T09:31:50Z

move storage implementation to storage directories




---
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] incubator-predictionio issue #349: [PIO-54] Avoid access key started with -

2017-02-26 Thread dszeto
Github user dszeto commented on the issue:

https://github.com/apache/incubator-predictionio/pull/349
  
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.
---


[jira] [Commented] (PIO-49) Add support for Elasticsearch 5.x

2017-02-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/PIO-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15884687#comment-15884687
 ] 

ASF GitHub Bot commented on PIO-49:
---

Github user marevol commented on the issue:

https://github.com/apache/incubator-predictionio/pull/336
  
Please see https://github.com/apache/incubator-predictionio/pull/352

- Move storage implementation to storage directory
- Put storage assembly JARs into lib/spark in distribution ZIP file
- To check ES1 support, default ES JAR in ZIP is for ES1 (please replace 
with ES5 before release)

If you have better location(lib/spark and lib/extra), please change them.


> Add support for Elasticsearch 5.x
> -
>
> Key: PIO-49
> URL: https://issues.apache.org/jira/browse/PIO-49
> Project: PredictionIO
>  Issue Type: Improvement
>Reporter: Shinsuke Sugaya
>
> We work on meta/event storage support for Elasticsearch 5.x.
> Although Elasticsearch 2.x does not allow dots in field names,
> Elasticsearch 5.x supports it. So, it's better to upgrade to ES 5.x release.
> Since ES 5.x provides Java Rest API client, we replaced
> Transport communication with HTTP one. Therefore, our fix
> uses HTTP(9200 port) only.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (PIO-49) Add support for Elasticsearch 5.x

2017-02-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/PIO-49?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15884684#comment-15884684
 ] 

ASF GitHub Bot commented on PIO-49:
---

GitHub user marevol opened a pull request:

https://github.com/apache/incubator-predictionio/pull/352

[PIO-49] Add support for Elasticsearch 5 and move storage implementation



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/marevol/incubator-predictionio move-storages

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-predictionio/pull/352.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #352


commit 3227fc7c270729df5e45fd069f93201851dd
Author: Shinsuke Sugaya 
Date:   2017-02-26T09:31:50Z

move storage implementation to storage directories




> Add support for Elasticsearch 5.x
> -
>
> Key: PIO-49
> URL: https://issues.apache.org/jira/browse/PIO-49
> Project: PredictionIO
>  Issue Type: Improvement
>Reporter: Shinsuke Sugaya
>
> We work on meta/event storage support for Elasticsearch 5.x.
> Although Elasticsearch 2.x does not allow dots in field names,
> Elasticsearch 5.x supports it. So, it's better to upgrade to ES 5.x release.
> Since ES 5.x provides Java Rest API client, we replaced
> Transport communication with HTTP one. Therefore, our fix
> uses HTTP(9200 port) only.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)