[GitHub] incubator-predictionio issue #195: ES 2.x

[pferrel <gi...@git.apache.org>]

Github user pferrel commented on the issue:

    https://github.com/apache/incubator-predictionio/pull/195
  
    @voran the reason this hasn't been merged is that ES 2.x libs are not built into Apache PIO. The previous version of ES in the 1.7.x line will need support for some time to come so we need to build for both. That means an sbt option for building 2 artifacts. 
    
    Also ES doesn't like the way sparkConf if used in engine.json so all Spark params must be possible to pass in on the command line. I think this works for all I know of so it means disabling a feature.
    
    @jramos has not updated this PR for Apache PIO so it would need a little work, mostly in changing the PIO namespace to org.apache.predictionio as per the conflicts above. So to use it with Apache PIO you'd need to make those mods.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---