hughhhh opened a new pull request, #43760:
URL: https://github.com/apache/superset/pull/43760
### SUMMARY
Fourth of four. Documents partition filter mapping for admins:
configuration, the operator matrix, and — deliberately — the parts a reader
will reasonably assume the feature covers but it does not.
Two sections carry most of the weight:
**"Transform preserves ordering"** explains why the checkbox exists at all,
with `hour()` as the worked example. It's a perfectly reasonable partition
transform on a `TIMESTAMP` column, and mirroring a time range through it
returns wrong numbers — `hour('2026-01-01 23:00')` is greater than
`hour('2026-01-02 01:00')` even though the first instant is earlier.
**"The assumption this rests on"** states the invariant plainly. Superset
emits a predicate on the partition column standing in for one on the mapped
column, which is only valid if `partition_column = transform(mapped_column)`
for every row. Superset cannot verify that — it's a property of whatever ETL
populates the partition column — and when it breaks the result is quietly wrong
charts rather than an error. The docs say so rather than letting people
discover it.
Also documented as explicit non-coverage, because each one will otherwise
surprise someone who expects "the dataset is now pruned": filter-value
dropdowns don't prune, RLS predicates and custom SQL `WHERE` clauses don't
mirror, and columns with an active advanced data type don't mirror.
`docs/static/feature-flags.json` was regenerated in PR 1, where the flag is
introduced.
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A — docs only.
### TESTING INSTRUCTIONS
```bash
cd docs && npm run build
```
Or read `docs/admin_docs/configuration/partition-filter-mapping.mdx`
directly. The page is picked up automatically by the auto-generated
`configuration` sidebar.
### ADDITIONAL INFORMATION
- [ ] Has associated issue:
- [ ] Required feature flags:
- [ ] Changes UI
- [ ] Includes DB Migration (follow approval process in
[SIP-59](https://github.com/apache/superset/issues/13351))
- [ ] 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]
