Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Raptor legacy connector #23588

Merged
merged 1 commit into from
Oct 1, 2024
Merged

Conversation

mosabua
Copy link
Member

@mosabua mosabua commented Sep 26, 2024

Description

The connector has no known use and forces some complexities on the rest of the codebase. Do allow next clean up steps the connector is removed.

In a contributor call, on slack, and in other conversations no concerns were expressed by anybody about the removal.

Additional context and related issues

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
(x) Release notes are required, with the following suggested text:

## Raptor connector
* {{breaking}} Remove the Raptor connector. ({issue}`23588`)

@cla-bot cla-bot bot added the cla-signed label Sep 26, 2024
@github-actions github-actions bot added the docs label Sep 26, 2024
@wendigo
Copy link
Contributor

wendigo commented Sep 26, 2024

YES!

@mosabua
Copy link
Member Author

mosabua commented Sep 27, 2024

Look at that .. all green. Not even failures from flaky, unrelated tests. I won the lotto...

@mosabua mosabua marked this pull request as ready for review September 27, 2024 04:35
@mosabua
Copy link
Member Author

mosabua commented Sep 27, 2024

Can I get an approval from @dain and/or @electrum ?

@mosabua
Copy link
Member Author

mosabua commented Sep 27, 2024

We requested user input on slack and linkedin to find out if anyone uses this plugin now.

@@ -41,15 +41,6 @@ public static OutputStreamOrcDataSink create(TrinoOutputFile outputFile)
return new OutputStreamOrcDataSink(outputFile.create(memoryContext), memoryContext);
}

// Do not use this method, it is here only for io.trino.plugin.raptor.legacy.storage.OrcFileWriter.createOrcDataSink
// and it should be removed in the future
@Deprecated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find

The connector has no known  users and forces some complexities
on the rest of the codebase. To allow this next clean up steps
the connector is removed.

In a contributor call, on slack, and in other conversations no concerns
were expressed by anybody about the removal.
@mosabua mosabua merged commit 61864c0 into trinodb:master Oct 1, 2024
5 of 13 checks passed
@mosabua mosabua deleted the raptor-rip branch October 1, 2024 15:34
@github-actions github-actions bot added this to the 460 milestone Oct 1, 2024
@wendigo
Copy link
Contributor

wendigo commented Oct 1, 2024

YEAH!

@mosabua mosabua mentioned this pull request Oct 1, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants