Sebastian Liebscher

build(docker): drop `build-essential` from Docker image

2

### SUMMARY This PR practically drops the apt package `build-essential` (`gcc` etc.) from the Docker image, by removing it in the same step it got installed. This package is needed...

refactor(logging): Clean up `statsd` logging

4

### SUMMARY This PR removes the `log_to_statsd=` functionality from the `@event_logger` and replaces usage with already existing `@statsd_metrics` decorator. I basically moved `BaseSupersetApiMixin` from `base_api.py` --> `base.py` so that the...

fix: correct value for config variable `UPLOAD_FOLDER`

2

### SUMMARY This PR fixes an incorrect default value for config variable `UPLOAD_FOLDER`: - current `superset/app/static/uploads/` - new `superset/static/uploads/` Additionally: - `static/uploads/` is now checked into git (analog to `superset/static/assets/`)...

chore: Remove redundant validation functions

1

### SUMMARY This PR removes redundant `validate_json` functions and replaces usage with already existing `superset.utils.schema`. ### TESTING INSTRUCTIONS - local unit / integration tests + GitHub CI ### ADDITIONAL INFORMATION...

chore: Cleanup function for stripping comments from SQL statements

2

### SUMMARY - `sql_parse.strip_comments_from_sql()` needed to instantiate `ParsedQuery` just to strip comments from a raw SQL string. Now it is the other way around, `ParsedQuery.strip_comments()` uses `strip_comments_from_sql()`. - DRY up...