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
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
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
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
Github user takezoe commented on the issue:
https://github.com/apache/incubator-predictionio/pull/391
https://github.com/apache/incubator-predictionio/pull/395
---
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
Github user takezoe commented on the issue:
https://github.com/apache/incubator-predictionio/pull/391
@pferrel Following methods have been deprecated since HBase 1.0.0. Since
they are still alive in HBase 1.2.6 current code works fine with 1.2.6.
I think it's not necessary to
Github user pferrel commented on the issue:
https://github.com/apache/incubator-predictionio/pull/391
Actually we already have an AWS AMI with HBase 1.2.6 that works fine
without any changes as far as we have seen and we do run automated tests.
Since this will require that
Github user takezoe commented on the issue:
https://github.com/apache/incubator-predictionio/pull/391
@dszeto Documentation updated.
---
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
Github user dszeto commented on the issue:
https://github.com/apache/incubator-predictionio/pull/391
@takezoe LGTM. Mind also updating the documentation in this PR to reflect
the new minimum requirement (http://predictionio.incubator.apache.org/install/)?
---
If your project is set