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

   ### SUMMARY
   
   YDB now has a dedicated sqlglot dialect provided by
   [ydb-sqlglot-plugin](https://pypi.org/project/ydb-sqlglot-plugin/),
   which registers itself automatically through sqlglot's `entry_points`
   mechanism (group `sqlglot.dialects`). Previously the `yql` engine was
   mapped to the ClickHouse dialect as a temporary stand-in, since no YDB
   dialect existed at the time the integration was added.
   
   Changes:
   - `SQLGLOT_DIALECTS["yql"]` switched from `Dialects.CLICKHOUSE` to `"ydb"`
   - `ydb-sqlglot-plugin` added to `YDBEngineSpec.metadata["pypi_packages"]` so 
the Superset UI lists it as a required package
   - `ydb-sqlglot-plugin` added to the `ydb` optional extras in `pyproject.toml`
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   
   N/A
   
   ### TESTING INSTRUCTIONS
   
   1. Install dependencies: `pip install "apache-superset[ydb]"`
   2. Configure a YDB database connection in Superset
   3. Run any YQL query — it should be parsed and executed correctly
   
   ### ADDITIONAL INFORMATION
   
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API


-- 
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