89e80e0cba
This teaches SQL to parse Elasticsearch's standard error responses but doesn't change SQL to general Elasticsearch's standard error responses in all cases. That can come in a followup. We do this parsing with jackson-core, the same dependency Elasticsearch uses for parsing json. We shade jackson-core in the JDBC driver so that users don't have to worry about dependency clashes. We do not do so in the CLI because it is a standalone application. We get a few "bonus" changes along the way: 1. We save a copy operation. Before this change responses were spooled into memory and then parsed. After this change they are parsed directly from the response stream. 2. We had a few classes entirely to support the spooling operation that we no longer need: `BytesArray`, `FastByteArrayInputStream`, and `BasicByteArrayOutputStream`. 3. SQL's `Version` was incorrectly parsing the version from the jar manifest. We didn't notice because the test was rigged to return `UNKNOWN` because we *were* running the test from the compiled classes directory instead of the jar. As part of shading jackson we moved running the tests to running against the shaded jar. Now we can actually assert that we parse the version correctly. It turns out we weren't. So I fixed it. Original commit: elastic/x-pack-elasticsearch@2e8f397bf4 |
||
|---|---|---|
| .. | ||
| elasticsearch-LICENSE.txt | ||
| elasticsearch-NOTICE.txt | ||
| jackson-LICENSE | ||
| jackson-NOTICE | ||
| jackson-core-2.8.10.jar.sha1 | ||
| jansi-1.16.jar.sha1 | ||
| jansi-LICENSE.txt | ||
| jansi-NOTICE.txt | ||
| jline-3.3.1.jar.sha1 | ||
| jline-LICENSE.txt | ||
| jline-NOTICE.txt | ||
| jna-4.4.0-1.jar.sha1 | ||
| jna-LICENSE.txt | ||
| jna-NOTICE.txt | ||