[GitHub] predictionio issue #466: [PIO-168] ES 6.X support + patch version dependency...

2018-10-06 Thread EmergentOrder
Github user EmergentOrder commented on the issue: https://github.com/apache/predictionio/pull/466 Not sure if that last failure was transient or what, but it looks like there are source dependencies on commons-lang3, so I added it to common/build.sbt. ---

[GitHub] predictionio issue #466: [PIO-168] ES 6.X support + patch version dependency...

2018-09-27 Thread EmergentOrder
Github user EmergentOrder commented on the issue: https://github.com/apache/predictionio/pull/466 Fixed the conflicts. I agree, looked like a transient error. Ready to merge, pending the next build. ---

[GitHub] predictionio issue #466: [PIO-168] ES 6.X support + patch version dependency...

2018-09-27 Thread EmergentOrder
Github user EmergentOrder commented on the issue: https://github.com/apache/predictionio/pull/466 Actually, scratch that, apps won't continue to work, although the indices will. We could add a fallback to the old indices in the code, but I think we should prefer to advise use

[GitHub] predictionio issue #466: [PIO-168] ES 6.X support + patch version dependency...

2018-09-27 Thread EmergentOrder
Github user EmergentOrder commented on the issue: https://github.com/apache/predictionio/pull/466 If users want to update to 6.X without the possibility to create new apps, but with existing apps preserved, they can use ES migration and PIO 0.13.0. ---

[GitHub] predictionio issue #466: [PIO-168] ES 6.X support + patch version dependency...

2018-09-27 Thread EmergentOrder
Github user EmergentOrder commented on the issue: https://github.com/apache/predictionio/pull/466 Actually, according to the docs, existing indices, and so apps, should continue to work in 6.X (but will be disallowed in 7.X). I assume there is a documented ES migration process. There