Skip to content

Commit 44c99e7

Browse files
authored
[native] Enable TaskUpdateRequest thrift serde in QueryRunner (prestodb#25857)
## Description <!---Describe your changes in detail--> Enable TaskUpdateRequest thrift serde in QueryRunner ## Motivation and Context <!---Why is this change required? What problem does it solve?--> <!---If it fixes an open issue, please link to the issue here.--> ## Impact <!---Describe any public API or user-facing feature change or any performance impact--> ## Test Plan <!---Please fill in how you tested your change--> ## Contributor checklist - [x] Please make sure your submission complies with our [contributing guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md), in particular [code style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style) and [commit standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards). - [x] PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced. - [x] Documented new properties (with its default value), SQL syntax, functions, or other functionality. - [x] If release notes are required, they follow the [release notes guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines). - [x] Adequate tests were added if applicable. - [x] CI passed. ## Release Notes Please follow [release notes guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines) and fill in the release notes below. ``` == NO RELEASE NOTE == ```
1 parent 61f6226 commit 44c99e7

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

presto-native-execution/src/test/java/com/facebook/presto/nativeworker/PrestoNativeQueryRunnerUtils.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -179,6 +179,8 @@ public HiveQueryRunnerBuilder setUseThrift(boolean useThrift)
179179
{
180180
this.extraProperties.put("experimental.internal-communication.thrift-transport-enabled", String.valueOf(useThrift));
181181
this.extraProperties.put("experimental.internal-communication.task-info-thrift-transport-enabled", String.valueOf(useThrift));
182+
this.extraProperties.put("experimental.internal-communication.task-update-request-thrift-serde-enabled", String.valueOf(useThrift));
183+
this.extraProperties.put("experimental.internal-communication.task-info-response-thrift-serde-enabled", String.valueOf(useThrift));
182184
return this;
183185
}
184186

0 commit comments

Comments
 (0)