Skip to content

Commit

Permalink
fix: add more clickhouse disallowed functions on config (#31198)
Browse files Browse the repository at this point in the history
  • Loading branch information
dpgaspar authored Dec 3, 2024
1 parent dd1ba96 commit 25f4226
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions UPDATING.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ assists people when migrating to a new version.

## Next

- [31198](https://github.com/apache/superset/pull/31198) Disallows by default the use of the following ClickHouse functions: "version", "currentDatabase", "hostName".
- [29798](https://github.com/apache/superset/pull/29798) Since 3.1.0, the intial schedule for an alert or report was mistakenly offset by the specified timezone's relation to UTC. The initial schedule should now begin at the correct time.
- [30021](https://github.com/apache/superset/pull/30021) The `dev` layer in our Dockerfile no long includes firefox binaries, only Chromium to reduce bloat/docker-build-time.
- [30099](https://github.com/apache/superset/pull/30099) Translations are no longer included in the default docker image builds. If your environment requires translations, you'll want to set the docker build arg `BUILD_TRANSACTION=true`.
Expand Down
2 changes: 1 addition & 1 deletion superset/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -1342,7 +1342,7 @@ def engine_context_manager( # pylint: disable=unused-argument
"table_to_xml_and_xmlschema",
"version",
},
"clickhouse": {"url"},
"clickhouse": {"url", "version", "currentDatabase", "hostName"},
"mysql": {"version"},
}

Expand Down

0 comments on commit 25f4226

Please sign in to comment.