Separate `include` parameter in `_filter_cols` into tags, ltypes, and column names
Đang mở
refactor
- Ngôn ngữ chính
- Python
- Star
- 155
- Fork
- 24
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
`WoodworkTableAccessor._filter_cols` is used by `select` and `describe`, and it takes in a single `include` parameter that tries to determine what is a semantic tag vs logical type vs column name. This has left us with some pretty complicated code that doesn't really do a perfect job.
We should separate the `include` parameter into `logical_types`, `semantic_tags`, and `column_names` to simplify things.
Hướng dẫn đóng góp
Đánh giá
Issue này chưa được đánh giá.