Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

"_explain" for SQL query that contains union is not valid json #114

Open
galkk opened this issue Jul 12, 2019 · 1 comment
Open

"_explain" for SQL query that contains union is not valid json #114

galkk opened this issue Jul 12, 2019 · 1 comment
Assignees
Labels
bug Something isn't working investigation low SQL

Comments

@galkk
Copy link
Contributor

galkk commented Jul 12, 2019

Currently explain request for union query returns invalid json.

example

@dai-chen
Copy link
Member

We added JSON pretty formatter in #166 which throws exception in this case.

[2019-10-31T15:39:23,593][WARN ][r.suppressed             ] [186590df9563.ant.amazon.com] path: /_opendistro/_sql/_explain, params: {pretty=}
com.fasterxml.jackson.core.JsonParseException: Unrecognized token 'performing': was expecting ('true', 'false' or 'null')
 at [Source: java.io.StringReader@5afbfe28; line: 1, column: 11]
	at com.fasterxml.jackson.core.JsonParser._constructError(JsonParser.java:1702) ~[jackson-core-2.8.11.jar:2.8.11]
	at com.fasterxml.jackson.core.base.ParserMinimalBase._reportError(ParserMinimalBase.java:558) ~[jackson-core-2.8.11.jar:2.8.11]
	at com.fasterxml.jackson.core.json.ReaderBasedJsonParser._reportInvalidToken(ReaderBasedJsonParser.java:2839) ~[jackson-core-2.8.11.jar:2.8.11]
	at com.fasterxml.jackson.core.json.ReaderBasedJsonParser._handleOddValue(ReaderBasedJsonParser.java:1903) ~[jackson-core-2.8.11.jar:2.8.11]
	at com.fasterxml.jackson.core.json.ReaderBasedJsonParser.nextToken(ReaderBasedJsonParser.java:749) ~[jackson-core-2.8.11.jar:2.8.11]
	at org.elasticsearch.common.xcontent.json.JsonXContentParser.nextToken(JsonXContentParser.java:52) ~[elasticsearch-x-content-7.3.1.jar:7.3.1]
	at org.elasticsearch.common.xcontent.json.JsonXContentGenerator.copyCurrentStructure(JsonXContentGenerator.java:415) ~[elasticsearch-x-content-7.3.1.jar:7.3.1]
	at org.elasticsearch.common.xcontent.XContentBuilder.copyCurrentStructure(XContentBuilder.java:988) ~[elasticsearch-x-content-7.3.1.jar:7.3.1]
	at com.amazon.opendistroforelasticsearch.sql.utils.JsonPrettyFormatter.format(JsonPrettyFormatter.java:45) ~[?:?]
	at com.amazon.opendistroforelasticsearch.sql.plugin.RestSqlAction.executeSqlRequest(RestSqlAction.java:156) ~[?:?]

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working investigation low SQL
Projects
No open projects
bugbash-April-2020
  
Awaiting triage
Development

No branches or pull requests

4 participants