sha174n opened a new pull request, #43020:
URL: https://github.com/apache/superset/pull/43020

   The PostgreSQL SQL Lab query validator shells out to the external `ecpg` 
binary (PostgreSQL's embedded-SQL C preprocessor) to check query syntax. That 
approach is fragile and heavy: it requires `ecpg` to be present in the runtime 
image, spawns an external preprocessor process for every validation request, 
and its behaviour varies across `ecpg`/PostgreSQL versions. The signal it 
provides is minimal next to the error a query returns when it runs.
   
   This removes the PostgreSQL validator and its default 
`SQL_VALIDATORS_BY_ENGINE` mapping. The Presto and SQLite validators are 
unchanged. Tests are updated accordingly.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to