Closed megri closed 8 years ago
Thanks for opening this. I have a fix in flight ready to go into master, but I'm waiting until #877 gets merged in because it affects the same location in the source.
Regarding issue #877 , I think the fix presented there is a red herring. The problem isn't with unencoded : or | but in fact the same problem outlined in this issue.
The proper fix:
;
should be URL-encoded for ES versions < 5|
edit: reference: https://www.elastic.co/guide/en/elasticsearch/reference/5.x/breaking_50_search_changes.html#_search_preferences
I recommend to not merge #877.
This should be fixed with this commit : https://github.com/elastic/elasticsearch-hadoop/commit/d442bb4479c2ae8478a10f25a7f0edb6b7256d87
What kind an issue is this?
The easier it is to track down the bug, the faster it is solved.
Often a solution already exists! Don’t send pull requests to implement new features without first getting our support. Sometimes we leave features out on purpose to keep the project small.
Issue description
This change results in
Steps to reproduce
Launch spark-shell with
./bin/spark-shell --packages --packages org.elasticsearch:elasticsearch-spark-20_2.11:5.0.0-rc1
NOTE Running with
org.elasticsearch:elasticsearch-spark-20_2.11:5.0.0-beta1
worksCode:
Strack trace:
Version Info
OS: : OS X Sierra JVM : Java HotSpot(TM) 64-Bit Server VM (build 25.102-b14, mixed mode) Hadoop/Spark: ?/2.0.1 ES-Hadoop : 5.0.0-rc1 ES : 2.4.1