[GitHub] incubator-predictionio-template-java-ecom-recommender issue #6: 0.12.0-incub...

2017-10-17 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio-template-java-ecom-recommender/pull/6 I have checked it works. @takezoe Can you review this PR? ---

[GitHub] incubator-predictionio-template-java-ecom-recommender pull request #6: 0.12....

2017-10-13 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio-template-java-ecom-recommender/pull/6 0.12.0-incubating release Oh, there isn't a development branch... use master Checking document... You can merge this pull request into a Git

[GitHub] incubator-predictionio-template-skeleton pull request #7: 0.12.0-incubating ...

2017-10-10 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio-template-skeleton/pull/7 0.12.0-incubating release Checking document... You can merge this pull request into a Git repository by running: $ git pull https://github.com/shimamoto

[GitHub] incubator-predictionio-template-attribute-based-classifier issue #12: 0.12.0...

2017-10-09 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio-template-attribute-based-classifier/pull/12 @takezoe Can you review this PR? ---

[GitHub] incubator-predictionio-template-similar-product issue #17: 0.12.0-incubating...

2017-10-09 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio-template-similar-product/pull/17 @takezoe Can you review this PR? ---

[GitHub] incubator-predictionio-template-ecom-recommender issue #14: 0.12.0-incubatin...

2017-10-09 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio-template-ecom-recommender/pull/14 @takezoe Can you review this PR? ---

[GitHub] incubator-predictionio issue #436: [PIO-125] Add Spark 2.2 support

2017-10-03 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/436 It is not supposed to need the new Guava dependency because Spark has Guava dependency. But if this dependency isn't included, compile error occurs at data project

Re: [VOTE] Resolution to create a TLP from graduating Incubator podling

2017-09-27 Thread takako shimamoto
n Soo <m...@apache.org> > * Luciano Resende <lrese...@apache.org> > * Marcin Ziemiński <zie...@apache.org> > * Marco Vivero <mviv...@apache.org> > * Mars Hall <m...@apache.org> > * Matthew Tovbin <tovb...@apache.or

Re: [ANNOUNCE] Apache PredictionIO 0.12.0-incubating Release

2017-09-27 Thread takako shimamoto
Awesome sauce! Chan, I owe you a lot. Thanks! 2017-09-28 12:45 GMT+09:00 Paritosh Piplewar : > congratulation.  > > Sent from my iPhone > > On 28-Sep-2017, at 3:10 AM, Chan Lee wrote: > > The Apache PredictionIO team would like to announce the

Re: [VOTE] Apache PredictionIO (incubating) 0.12.0 Release (RC3)

2017-09-19 Thread takako shimamoto
+1 binding I checked: - Binaries working - Test the official templates (Recommendation, E-Commerce Recommendation, Similar Product and Classification) using Maven artifacts Much appreciated, Chan! 2017-09-19 6:31 GMT+09:00 Chan Lee : > +1 binding > > > On Mon, Sep 18,

[GitHub] incubator-predictionio-template-recommender pull request #19: [WIP] 0.12.0-i...

2017-09-15 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio-template-recommender/pull/19 [WIP] 0.12.0-incubating release waiting for official 0.12.0 release You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-predictionio-template-ecom-recommender pull request #14: [WIP] 0.1...

2017-09-15 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio-template-ecom-recommender/pull/14 [WIP] 0.12.0-incubating release waiting for official 0.12.0 release You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-predictionio-template-similar-product pull request #17: [WIP] 0.12...

2017-09-15 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio-template-similar-product/pull/17 [WIP] 0.12.0-incubating release waiting for official 0.12.0 release You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-predictionio-template-attribute-based-classifier pull request #12:...

2017-09-15 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio-template-attribute-based-classifier/pull/12 [WIP] 0.12.0-incubating release waiting for official 0.12.0 release You can merge this pull request into a Git repository by running

Re: Graduation to TLP

2017-09-07 Thread takako shimamoto
t >>> graduation. We are also asking that people check things like the Maturity >>> Checklist to see it we are ready. http://community.apache.org/ >>> apache-way/apache-project-maturity-model.html < >>> http://community.apache.org/apache-way/apache-proje

[GitHub] incubator-predictionio issue #426: Fix use of case class in textclassifier e...

2017-08-28 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/426 This fix is LGTM! But I wonder what this menu is... 🤔 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-predictionio-template-text-classifier issue #14: Fix use of case c...

2017-08-28 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio-template-text-classifier/pull/14 LGTM! --- 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

[GitHub] incubator-predictionio pull request #425: [PIO-110] Refactoring

2017-08-25 Thread shimamoto
Github user shimamoto commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/425#discussion_r135187599 --- Diff: storage/hbase/src/main/scala/org/apache/predictionio/data/storage/hbase/HBEventsUtil.scala --- @@ -376,32 +375,30 @@ object

[GitHub] incubator-predictionio pull request #425: [PIO-110] Refactoring

2017-08-25 Thread shimamoto
Github user shimamoto commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/425#discussion_r135181421 --- Diff: data/src/main/scala/org/apache/predictionio/data/api/Webhooks.scala --- @@ -115,22 +107,22 @@ private[predictionio] object

[GitHub] incubator-predictionio pull request #425: [PIO-110] Refactoring

2017-08-25 Thread shimamoto
Github user shimamoto commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/425#discussion_r135187633 --- Diff: storage/hbase/src/main/scala/org/apache/predictionio/data/storage/hbase/HBEventsUtil.scala --- @@ -376,32 +375,30 @@ object

[GitHub] incubator-predictionio pull request #425: [PIO-110] Refactoring

2017-08-25 Thread shimamoto
Github user shimamoto commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/425#discussion_r135191103 --- Diff: data/src/main/scala/org/apache/predictionio/data/api/Webhooks.scala --- @@ -115,22 +107,22 @@ private[predictionio] object

[GitHub] incubator-predictionio pull request #425: [PIO-110] Refactoring

2017-08-25 Thread shimamoto
Github user shimamoto commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/425#discussion_r135175043 --- Diff: core/src/main/scala/org/apache/predictionio/controller/PAlgorithm.scala --- @@ -115,15 +115,12 @@ abstract class PAlgorithm[PD

[GitHub] incubator-predictionio pull request #425: [PIO-110] Refactoring

2017-08-25 Thread shimamoto
Github user shimamoto commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/425#discussion_r135180795 --- Diff: data/src/main/scala/org/apache/predictionio/data/api/Webhooks.scala --- @@ -62,22 +53,23 @@ private[predictionio] object

[GitHub] incubator-predictionio issue #425: [PIO-110] Refactoring

2017-08-24 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/425 @takezoe Will do. --- 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

Re: August 2017 Release

2017-08-03 Thread takako shimamoto
I think it's almost ready, and now we just have to update the current documentation. The deadline of several unresolved issues for Target Version/s: 0.12.0-incubating is extended, right? 2017-08-01 5:59 GMT+09:00 Donald Szeto : > Hey all, > > To adhere to the proposed release

[GitHub] incubator-predictionio issue #417: [PIO-66] JIRA and release process for Pre...

2017-08-02 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/417 Currently there isn't section to put documents for committers. It is better to clean up not only Resources section but also the others. It might be a good idea to do it in another

[GitHub] incubator-predictionio pull request #417: [PIO-66] JIRA and release process ...

2017-07-31 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio/pull/417 [PIO-66] JIRA and release process for PredictionIO https://issues.apache.org/jira/browse/PIO-66?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel You can merge

[GitHub] incubator-predictionio issue #405: [PIO-63]Add incubator logo and disclaimer...

2017-07-28 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/405 @dszeto I also would like to hear your opinion. --- 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

[jira] [Commented] (PIO-66) Document JIRA processes and add to public documentation

2017-07-27 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-66?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16104429#comment-16104429 ] Takako Shimamoto commented on PIO-66: - remaining items: * Add to public documentation I’ll take care

[jira] [Resolved] (PIO-60) Fix minor issues with author information

2017-07-27 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-60?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Takako Shimamoto resolved PIO-60. - Resolution: Fixed Fix Version/s: 0.12.0-incubating > Fix minor issues with author informat

[jira] [Updated] (PIO-86) Maintain a working docker container

2017-07-27 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-86?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Takako Shimamoto updated PIO-86: Target Version/s: (was: 0.12.0-incubating) > Maintain a working docker contai

[jira] [Commented] (PIO-86) Maintain a working docker container

2017-07-27 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-86?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16104397#comment-16104397 ] Takako Shimamoto commented on PIO-86: - That makes sense to me. This topic isn't related to the apache

[jira] [Closed] (PIO-87) Fix doc licensing issues

2017-07-27 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-87?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Takako Shimamoto closed PIO-87. --- Resolution: Duplicate Let us open another ticket if necessary. > Fix doc licensing iss

[jira] [Resolved] (PIO-62) Investigate RAT issues and other authorship information

2017-07-27 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-62?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Takako Shimamoto resolved PIO-62. - Resolution: Fixed Fix Version/s: 0.12.0-incubating > Investigate RAT issues and ot

[jira] [Closed] (PIO-9) Clean up examples under examples/

2017-07-25 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-9?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Takako Shimamoto closed PIO-9. -- Resolution: Fixed Completed. > Clean up examples under examp

[jira] [Closed] (PIO-107) Removal of examples under examples/experimental

2017-07-24 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-107?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Takako Shimamoto closed PIO-107. > Removal of examples under examples/experimen

[GitHub] incubator-predictionio pull request #416: [PIO-107] Removal of examples unde...

2017-07-23 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio/pull/416 [PIO-107] Removal of examples under examples/experimental You can merge this pull request into a Git repository by running: $ git pull https://github.com/shimamoto/incubator

[jira] [Created] (PIO-107) Removal of examples under examples/experimental

2017-07-23 Thread Takako Shimamoto (JIRA)
Takako Shimamoto created PIO-107: Summary: Removal of examples under examples/experimental Key: PIO-107 URL: https://issues.apache.org/jira/browse/PIO-107 Project: PredictionIO Issue Type

[GitHub] incubator-predictionio issue #409: Fix use of case class in recommendation e...

2017-07-14 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/409 LGTM! --- 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

[GitHub] incubator-predictionio issue #411: Fix use of case class in classification e...

2017-07-14 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/411 LGTM! --- 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

[GitHub] incubator-predictionio issue #408: Fix use of case class in recommendation e...

2017-07-14 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/408 LGTM! --- 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

[GitHub] incubator-predictionio-template-recommender issue #18: Fix use of case class

2017-07-14 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio-template-recommender/pull/18 LGTM! --- 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

[jira] [Closed] (PIO-97) Fixes examples of the official templates

2017-07-09 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-97?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Takako Shimamoto closed PIO-97. --- Resolution: Fixed I'm done! > Fixes examples of the official templa

Re: [jira] [Commented] (PIO-97) Fixes examples of the official templates

2017-07-09 Thread takako shimamoto
of the official templates >> > >> > >> > Key: PIO-97 >> > URL: https://issues.apache.org/jira/browse/PIO-97 >> > Project: PredictionIO >> > Issue Type: Sub-task >> &

[GitHub] incubator-predictionio issue #400: [PIO-97] Fixes examples of the official t...

2017-07-06 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/400 We ( @takezoe helped me ) confirmed everything works. Unless we hear from you to the contrary, we will merge this next week. --- If your project is set up for it, you can reply

[GitHub] incubator-predictionio pull request #389: [PIO-91] Fixed hadoop-hdfs artifac...

2017-06-05 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio/pull/389 [PIO-91] Fixed hadoop-hdfs artifact missing error I made a mistake when I reviewed dependencies. Basically, the problem is due to unavailability of the hadoop-hdfs jars. You

[jira] [Assigned] (PIO-91) hadoop-hdfs artifact missing when creating binary version of PredictionIO

2017-06-04 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-91?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Takako Shimamoto reassigned PIO-91: --- Assignee: Takako Shimamoto > hadoop-hdfs artifact missing when creating binary vers

[jira] [Commented] (PIO-91) hadoop-hdfs artifact missing when creating binary version of PredictionIO

2017-05-31 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-91?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16032327#comment-16032327 ] Takako Shimamoto commented on PIO-91: - Thanks for reporting this problem. I maybe made a mistake when I

Re: New PMC member and committer: Takako Shimamoto

2017-05-18 Thread takako shimamoto
; wrote: >> >> Hi all, >> >> The Project Management Committee (PMC) for Apache PredictionIO >> (incubating) >> has asked Takako Shimamoto to become a PMC member and committer, and we >> are >> pleased to announce that she has accepted. >> >>

[jira] [Commented] (PIO-31) Move from spray to akka-http in servers

2017-05-16 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-31?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16013515#comment-16013515 ] Takako Shimamoto commented on PIO-31: - I agree with Naoki. The cost of supporting multiple versions

[GitHub] incubator-predictionio issue #382: [PIO-64] Fixed a bug that JDBCPEvents uni...

2017-05-15 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/382 @EmergentOrder Thanks for review. I’m glad things worked out fine :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[jira] [Commented] (PIO-64) JDBCPEvents unit tests fail intermittently

2017-05-10 Thread Takako Shimamoto (JIRA)
[ https://issues.apache.org/jira/browse/PIO-64?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16004528#comment-16004528 ] Takako Shimamoto commented on PIO-64: - Anyway, I just gave it a try. https://github.com/shimamoto

Re: Request for Integration of Issue Management

2017-04-09 Thread takako shimamoto
ance to look at everyone of > them yet. > > Any help would be appreciated. Perhaps there's a way to migrate all of them > to JIRA first and examine later? > > On Fri, Apr 7, 2017 at 4:53 AM takako shimamoto <chiboch...@gmail.com> > wrote: > >> Hi All, >> >&g

Request for Integration of Issue Management

2017-04-07 Thread takako shimamoto
Hi All, I think it would be better to integrate issues management. In particular, it is hard for new users to understand whether issues are managed by JIRA or GitHub. For example in case of Spark or Hadoop, which are other Apache Project, users cannot use GitHub Issues. I was hoping that you

[GitHub] incubator-predictionio pull request #365: Remove duplicate settings.

2017-03-23 Thread shimamoto
Github user shimamoto commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/365#discussion_r107589818 --- Diff: tools/build.sbt --- @@ -27,8 +27,6 @@ libraryDependencies ++= Seq( "io.spray" %% "spray-tes

[GitHub] incubator-predictionio pull request #365: Remove duplicate settings.

2017-03-23 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio/pull/365 Remove duplicate settings. Thanks to merge some of #364 changes. However, since there are places where it looks some mistakes (duplication etc.), I fixed it. You can merge

[GitHub] incubator-predictionio issue #364: [PIO-30] Spark 2 support on sbt way

2017-03-20 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/364 @dszeto I understand your situation. This PR is put EoM (Ease of Maintenance) before flexible because this leads to Easy of Extension in future. But as you say that the most

[GitHub] incubator-predictionio pull request #364: [PIO-30] Spark 2 support on sbt wa...

2017-03-20 Thread shimamoto
Github user shimamoto closed the pull request at: https://github.com/apache/incubator-predictionio/pull/364 --- 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

[GitHub] incubator-predictionio pull request #364: [PIO-30] Spark 2 support on sbt wa...

2017-03-17 Thread shimamoto
GitHub user shimamoto opened a pull request: https://github.com/apache/incubator-predictionio/pull/364 [PIO-30] Spark 2 support on sbt way The goal of this PR is: - Define the versions of Scala to build against in the crossScalaVersions setting - Scala 2.10 and 2.11

[GitHub] incubator-predictionio issue #345: [PIO-30] Set up a cross build for Spark 2...

2017-03-02 Thread shimamoto
Github user shimamoto commented on the issue: https://github.com/apache/incubator-predictionio/pull/345 Hi @chanlee514 I am interested in Spark 2.x support. But this PR bothers me a little. Do you mean that the true nature of this PR is that we can choose either Spark 1.6