gateway icon indicating copy to clipboard operation
gateway copied to clipboard

Published 20 hours ago • verified publisher icon Portkey-AI

[Feature] Valid SQL PLugin

Open siddharthsambharia-portkey opened this issue 1 year ago • 5 comments

What Would You Like to See with the Gateway?

Plugin Name: Valid SQL Plugin Description: Validates whether the given SQL code is syntactically correct

Here is the Contributing Guide for Plugin

siddharthsambharia-portkey avatar Aug 16 '24 05:08 siddharthsambharia-portkey

@VisargD can i work on this

keshavkrishna avatar Sep 26 '24 18:09 keshavkrishna

Sure. Assigning the issue to you. Thanks!

VisargD avatar Sep 26 '24 18:09 VisargD

@vrushankportkey Looks like no activity here as well, can I take this up?

Question: What is the scope of this, do we consider about DB specific syntaxes here? or just a regular SQL only. And is it okay to use js sql parser kind of package for this.

I'm thinking to use node-sql-parser, as it comes with support of 13 DBs. But looks like bundle size is bit heavier: image.

5war00p avatar Oct 31 '24 15:10 5war00p

Hi @5war00p please take it up! Yes we would like the bundle size to be as small as possible. @VisargD thoughts?

vrushankportkey avatar Nov 01 '24 05:11 vrushankportkey

Yeah makes sense, I'm not sure whether it is tree-shakable or not. We should validate that as well, if we are choosing some package.

5war00p avatar Nov 01 '24 05:11 5war00p