Skip to content

Commit 621b86d

Browse files
committed
Fix product tests
- remove airline from classpath by excluding it from com.teradata.tpcds:tpcds - force presto-product-tests to shade airline by disabling the generation of the dependency-reduced-pom - change message() to body() in StatementClient to get correct error messages
1 parent fcef004 commit 621b86d

File tree

3 files changed

+6
-1
lines changed

3 files changed

+6
-1
lines changed

pom.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1312,6 +1312,10 @@
13121312
<groupId>com.google.code.findbugs</groupId>
13131313
<artifactId>annotations</artifactId>
13141314
</exclusion>
1315+
<exclusion>
1316+
<groupId>io.airlift</groupId>
1317+
<artifactId>airline</artifactId>
1318+
</exclusion>
13151319
</exclusions>
13161320
</dependency>
13171321

presto-cli/pom.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,7 @@
142142
<configuration>
143143
<shadedArtifactAttached>true</shadedArtifactAttached>
144144
<shadedClassifierName>executable</shadedClassifierName>
145+
<createDependencyReducedPom>false</createDependencyReducedPom>
145146
<transformers>
146147
<transformer implementation="org.apache.maven.plugins.shade.resource.ManifestResourceTransformer">
147148
<manifestEntries>

presto-client/src/main/java/com/facebook/presto/client/StatementClientV1.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -498,7 +498,7 @@ private RuntimeException requestFailedException(String task, Request request, Js
498498
if (!response.hasValue()) {
499499
if (response.getStatusCode() == HTTP_UNAUTHORIZED) {
500500
return new ClientException("Authentication failed" +
501-
Optional.ofNullable(response.getStatusMessage())
501+
Optional.ofNullable(response.getResponseBody())
502502
.map(message -> ": " + message)
503503
.orElse(""));
504504
}

0 commit comments

Comments
 (0)