Skip to content

Commit 4427621

Browse files
fix: PR comments
1 parent 461025b commit 4427621

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

docs/how/updating-datahub.md

+2-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,8 @@ This file documents any backwards-incompatible changes in DataHub and assists pe
4949
- #11570 - The `DatahubClientConfig`'s server field no longer defaults to `http://localhost:8080`. Be sure to explicitly set this.
5050
- #11570 - If a `datahub_api` is explicitly passed to a stateful ingestion config provider, it will be used. We previously ignored it if the pipeline context also had a graph object.
5151
- #11518 - DataHub Garbage Collection: Various entities that are soft-deleted (after 10d) or are timeseries _entities_ (dataprocess, execution requests) will be removed automatically using logic in the `datahub-gc` ingestion source.
52-
- #12020 - Removed the `sqlparse` and `sqllineage` dependencies from ingestion and eliminated the `sql_parser` configuration from the Redash source, as Redash now exclusively uses the SQLGlot-based parser for lineage extraction.
52+
- #12020 - Removed `sql_parser` configuration from the Redash source, as Redash now exclusively uses the sqlglot-based parser for lineage extraction.
53+
- #12020 - Removed `datahub.utilities.sql_parser`, `datahub.utilities.sql_parser_base` and `datahub.utilities.sql_lineage_parser_impl` module along with `SqlLineageSQLParser` and `DefaultSQLParser`. Use `create_lineage_sql_parsed_result` from `datahub.sql_parsing.sqlglot_lineage` module instead.
5354

5455
### Potential Downtime
5556

metadata-ingestion/docs/sources/redash/redash.md

-5
This file was deleted.

0 commit comments

Comments
 (0)