Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Subtask] Develop Generic Expression system #674

Closed
Tracked by #600
mchades opened this issue Nov 6, 2023 · 0 comments · Fixed by #721
Closed
Tracked by #600

[Subtask] Develop Generic Expression system #674

mchades opened this issue Nov 6, 2023 · 0 comments · Fixed by #721
Assignees
@mchades

Comments

@mchades
Copy link
Contributor

mchades commented Nov 6, 2023

Describe the subtask

As the design of the generic Expression system was completed, the development work should be carried out.

Parent issue

#600
@mchades mchades self-assigned this Nov 6, 2023
This was referenced Nov 6, 2023
Closed
Merged
jerryshao pushed a commit that referenced this issue Nov 17, 2023
@mchades
[#674] feat(all): add a unified Expression system (#721)
c11c996 
### What changes were proposed in this pull request?

- Add a unified Expression system
- refactor expression of partitioning, sortOrder, and distribution

### Why are the changes needed?

clarify Expression, Partitioning, SortOrder, and Distribution

Fix: #674 

### Does this PR introduce _any_ user-facing change?

Yep. There have been changes in the representation of Partitioning,
SortOrderDTO, and DistributionDTO.

### How was this patch tested?

existing tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mchades mchades

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#674] feat(all): add a unified Expression system mchades/graviton
1 participant
@mchades