rusackas opened a new pull request, #38174: URL: https://github.com/apache/superset/pull/38174
## Summary When using the ODPS (MaxCompute) data source, previewing partitioned tables in SQLLab would fail because ODPS requires a partition to be specified for partition tables. This PR adds ODPS-specific handling: - New `OdpsEngineSpec` with partition detection support - Modified `select_star` to add partition filter for ODPS partition tables - New `Partition` dataclass in `sql/parse.py` - New `is_odps_partitioned_table` method in `DatabaseDAO` **This is an adoption of #32556 by @zhutong6688**, rebased on current master with: - Fixed MyPy error (removed non-existent `allow_limit_clause` attribute) - Updated typing to modern Python style (`list[str] | None` instead of `Optional[List[str]]`) - Code style improvements Closes #32301 Supersedes #32556 ## Test plan - [ ] Verify ODPS partition table preview works in SQLLab - [ ] Verify non-partitioned ODPS tables still work - [ ] Verify other database engines are unaffected 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: zhutong6688 <[email protected]> -- 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]
