Skip to content

Remove legacy task info thrift serde#25464

Merged
arhimondr merged 2 commits into
prestodb:masterfrom
shangm2:removeLegacyTaskinfoSerde
Jul 5, 2025
Merged

Remove legacy task info thrift serde#25464
arhimondr merged 2 commits into
prestodb:masterfrom
shangm2:removeLegacyTaskinfoSerde

Conversation

@shangm2

@shangm2 shangm2 commented Jun 30, 2025

Copy link
Copy Markdown
Contributor

Description

  1. Remove legacy task info thrift serde since we are having the new one

Motivation and Context

Impact

Test Plan

  1. Testing

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

== NO RELEASE NOTE ==

@shangm2 shangm2 requested a review from a team as a code owner June 30, 2025 20:41
@prestodb-ci prestodb-ci added the from:Meta PR from Meta label Jun 30, 2025
@shangm2 shangm2 force-pushed the removeLegacyTaskinfoSerde branch from 2814b61 to 4046f4b Compare June 30, 2025 22:02
@shangm2 shangm2 changed the title remove legacy task info thrift serde Remove legacy task info thrift serde Jun 30, 2025
@shangm2 shangm2 force-pushed the removeLegacyTaskinfoSerde branch from 162ed5d to a128dc6 Compare July 1, 2025 23:44
@shangm2 shangm2 force-pushed the removeLegacyTaskinfoSerde branch from a128dc6 to 8da9576 Compare July 1, 2025 23:46
@arhimondr arhimondr merged commit 55259e5 into prestodb:master Jul 5, 2025
108 checks passed
shangm2 pushed a commit that referenced this pull request Jul 15, 2025
## Description
- Native changes to enable thrift for getTaskInfo and deleteTask
endpoints
- Depends on #25464

## Motivation and Context
prestodb/rfcs#38

## Test Plan
Build package and test in verifier: 230498

## 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 ==
```
lga-zurich pushed a commit to lga-zurich/presto-exchange that referenced this pull request Sep 2, 2025
## Description
- Native changes to enable thrift for getTaskInfo and deleteTask
endpoints
- Depends on prestodb#25464

## Motivation and Context
prestodb/rfcs#38

## Test Plan
Build package and test in verifier: 230498

## 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 ==
```
lga-zurich pushed a commit to lga-zurich/presto-exchange that referenced this pull request Sep 8, 2025
## Description
- Native changes to enable thrift for getTaskInfo and deleteTask
endpoints
- Depends on prestodb#25464

## Motivation and Context
prestodb/rfcs#38

## Test Plan
Build package and test in verifier: 230498

## 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 ==
```
lga-zurich pushed a commit to lga-zurich/presto-exchange that referenced this pull request Sep 8, 2025
## Description
- Native changes to enable thrift for getTaskInfo and deleteTask
endpoints
- Depends on prestodb#25464

## Motivation and Context
prestodb/rfcs#38

## Test Plan
Build package and test in verifier: 230498

## 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 ==
```
lga-zurich pushed a commit to lga-zurich/presto-exchange that referenced this pull request Sep 8, 2025
## Description
- Native changes to enable thrift for getTaskInfo and deleteTask
endpoints
- Depends on prestodb#25464

## Motivation and Context
prestodb/rfcs#38

## Test Plan
Build package and test in verifier: 230498

## 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 ==
```
lga-zurich pushed a commit to lga-zurich/presto-exchange that referenced this pull request Sep 8, 2025
## Description
- Native changes to enable thrift for getTaskInfo and deleteTask
endpoints
- Depends on prestodb#25464

## Motivation and Context
prestodb/rfcs#38

## Test Plan
Build package and test in verifier: 230498

## 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 ==
```
lga-zurich pushed a commit to lga-zurich/presto-exchange that referenced this pull request Sep 8, 2025
## Description
- Native changes to enable thrift for getTaskInfo and deleteTask
endpoints
- Depends on prestodb#25464

## Motivation and Context
prestodb/rfcs#38

## Test Plan
Build package and test in verifier: 230498

## 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 ==
```
lga-zurich pushed a commit to lga-zurich/presto-exchange that referenced this pull request Sep 8, 2025
## Description
- Native changes to enable thrift for getTaskInfo and deleteTask
endpoints
- Depends on prestodb#25464

## Motivation and Context
prestodb/rfcs#38

## Test Plan
Build package and test in verifier: 230498

## 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 ==
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

from:Meta PR from Meta

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants