The requirements.txt file lists `5.5.3` as the minimum version for 
elasticsearch. Official SQL support landed in elasticsearch as of `6.3.0`.  It 
seems like this PR is still predicated on 5.5.3? Much like its 
[predecessor](https://github.com/apache/incubator-superset/pull/2953)? With 
perhaps an attempt to map the SQL coming in the SQL alchemy api without relying 
on the new sql[ _translate 
endpoint](https://www.elastic.co/guide/en/elasticsearch/reference/current/sql-translate.html)?
 If that's the intent here, I think it would be smart to focus solely on ES 
6.3.0+ -- am I understanding this correctly?

[ Full content available at: 
https://github.com/apache/incubator-superset/pull/5358 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org

Reply via email to