[GitHub] incubator-predictionio issue #391: [PIO-92] Bump HBase to 1.2.6 (and fix dep...

2017-06-27 Thread takezoe
Github user takezoe commented on the issue:

https://github.com/apache/incubator-predictionio/pull/391
  
@dszeto Thanks for your support. I updated docs in 
e3f8307891ae0a5b287855ad83c41379c4e1e8bc so close this pull request and 
[PIO-92](https://issues.apache.org/jira/projects/PIO/issues/PIO-92). I will 
raise a new ticket to clarify supported versions and recommended versions in 
docs, and continue to work on that ticket.


---
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 #391: [PIO-92] Bump HBase to 1.2.6 (and fix dep...

2017-06-27 Thread dszeto
Github user dszeto commented on the issue:

https://github.com/apache/incubator-predictionio/pull/391
  
Sorry for the late reply. Agree we should start properly deprecating it. 
Regarding the inconsistency, I think we should correct `0.98.6 in 
https://predictionio.incubator.apache.org/install/`, and indicate right now 
0.98.5 is the minimum compatible version, but it is deprecated. I think it is 
okay to keep a more recent HBase version in the installation instructions 
though, so that new users do not install an EOMed version of HBase. Thoughts?


---
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 #391: [PIO-92] Bump HBase to 1.2.6 (and fix dep...

2017-06-26 Thread takezoe
Github user takezoe commented on the issue:

https://github.com/apache/incubator-predictionio/pull/391
  
@dszeto @pferrel How about keeping current code but update the 
prerequisites section of installation guide to clarify supported versions?

About HBase, PIO supports 0.98.5 or higher, I will make a table of 
supported versions in the prerequisites section of document and write this in 
that table with other software versions.
In addition, I would like to update the installation guide to use the 
newest supported version (1.2.6 in HBase) for all softwares as well as HBase.

Is this alright?


---
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 #391: [PIO-92] Bump HBase to 1.2.6 (and fix dep...

2017-06-21 Thread takezoe
Github user takezoe commented on the issue:

https://github.com/apache/incubator-predictionio/pull/391
  
@pferrel and I found 3 versions of HBase at least in `build.sbt` and 
documentation, so I'm not sure what versions are supported in the current 
PredictionIIO.

- 0.98.5 in 
https://github.com/apache/incubator-predictionio/blob/develop/storage/hbase/build.sbt
- 0.98.6 in https://predictionio.incubator.apache.org/install/
- 1.2.5 in 
https://predictionio.incubator.apache.org/install/install-sourcecode/

It is inconsistent. What's state that should be originally?


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