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

Move simple classes after refactoring to sql-plugin-api #9611

Merged

Conversation

gerashegalov
Copy link
Collaborator

@gerashegalov gerashegalov commented Nov 2, 2023

I noticed a couple more classes that can be easily moved to sql-plugin-api after refactoring

Contributes to #6565

  • RapidsUDF
  • ExclusiveModeGpuDiscoveryPlugin
  • Retryable
  • udf.Plugin
  • SQLPlugin

to sql-plugin-api

Signed-off-by: Gera Shegalov [email protected]

- RapidsUDF
- ExclusiveModeGpuDiscoveryPlugin

Signed-off-by: Gera Shegalov <[email protected]>
@gerashegalov gerashegalov changed the title Move RapidsUDF, ExclusiveModeGpuDiscoveryPlugin to sql-plugin-api Move RapidsUDF, ExclusiveModeGpuDiscoveryPlugin, Retryable to sql-plugin-api Nov 2, 2023
@gerashegalov gerashegalov self-assigned this Nov 2, 2023
@gerashegalov gerashegalov marked this pull request as draft November 2, 2023 17:32
@gerashegalov gerashegalov changed the title Move RapidsUDF, ExclusiveModeGpuDiscoveryPlugin, Retryable to sql-plugin-api Move simple classes after refactoring to sql-plugin-api Nov 2, 2023
Signed-off-by: Gera Shegalov <[email protected]>
@gerashegalov
Copy link
Collaborator Author

build

@gerashegalov gerashegalov marked this pull request as ready for review November 2, 2023 21:33
@gerashegalov gerashegalov merged commit 614f873 into NVIDIA:branch-23.12 Nov 3, 2023
37 checks passed
@gerashegalov gerashegalov deleted the moveSomethingToSQLAPI-3 branch November 3, 2023 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants