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

vdk-smarter: introduce simple open ai plugin #2351

Merged
merged 3 commits into from
Jul 7, 2023

Conversation

antoniivanov
Copy link
Collaborator

@antoniivanov antoniivanov commented Jul 3, 2023

Very simple plugin that reviews every query and scores it using OpenAI models. The purpose is to provide easy visibility into queries and their quality for the user. Though technically they could potentially use chatGPT themselves or static code analysis, that is less likely and and harder for them

A) because some queries are generated dynamically
B) Many queries could be part of a job and take (development) time to check.
C) the prompt engineering is something that takes time as well and tuning as well. The prompts could be tuned by community pretty well over time.

Inside an enterprise, this enables central data team to establish best practices around SQL with proper review model themselves as well.

This is a POC (a Pre-Alpha development status as defined in https://martin-thoma.com/software-development-stages/#2-pre-alpha)

Very simple plugin that reviews every query and scores it using OpenAI
models. The purpose is to provide easy visibility into queries and their
quality for the user. Though technically they could potentially use
chatGPT themselves or static code analysis , that is less likely and
alsl generally harder

A) because some queries are generated dynamically
B) lots of queries could be part of a job and takes time to check all.
C) the prompt engineering is something that takes time as well and
tuning as well. The prompts could be tuned by community pretty well
over time.

This enables central data team to establish best practices around SQL
with proper review model .

Signed-off-by: Antoni Ivanov <aivanov@vmware.com>
@antoniivanov antoniivanov force-pushed the person/aivanov/vdk-smarter branch from 4738e0c to fca3eb1 Compare July 5, 2023 12:35
Signed-off-by: Antoni Ivanov <aivanov@vmware.com>
@antoniivanov antoniivanov force-pushed the person/aivanov/vdk-smarter branch from fca3eb1 to 3056b41 Compare July 5, 2023 13:03
Copy link
Collaborator

@duyguHsnHsn duyguHsnHsn left a comment

Choose a reason for hiding this comment

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

Looks good to me! Shouldn't be there a VEP for this?

@antoniivanov
Copy link
Collaborator Author

Looks good to me! Shouldn't be there a VEP for this?

Not at this stage. This is just POC (note the development status of the plugin Pre-Alpha).

@antoniivanov antoniivanov merged commit ecc111a into main Jul 7, 2023
@antoniivanov antoniivanov deleted the person/aivanov/vdk-smarter branch July 7, 2023 11:38
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.

4 participants