Skip to content

Introduce dedicated type representing variables within an aggregation. #4242

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
Closed
wants to merge 2 commits into from
Closed

Introduce dedicated type representing variables within an aggregation. #4242

wants to merge 2 commits into from

Conversation

christophstrobl
Copy link
Member

@christophstrobl christophstrobl commented Nov 30, 2022 •
edited
Loading

This PR introduces a new AggregationVariable type that is intended to better identify variables within a pipeline to avoid mapping failures caused by invalid field names.

Closes: #4070

@christophstrobl
Prepare issue branch.
d2feb45
@christophstrobl
Introduce AggregationVariable type.
64909e6 
This commit introduces a new AggregationVariable type that is intended to better identify variables within a pipeline to avoid mapping failures caused by invalid field names.
@mp911de mp911de added type: enhancement A general enhancement type: bug A general bug and removed type: enhancement A general enhancement labels Aug 17, 2023
@mp911de mp911de closed this Aug 17, 2023
mp911de pushed a commit that referenced this pull request Aug 17, 2023
@christophstrobl @mp911de
Introduce AggregationVariable type.
f1cff3c 
This commit introduces a new AggregationVariable type that is intended to better identify variables within a pipeline to avoid mapping failures caused by invalid field names.

Closes #4070
Original pull request: #4242
mp911de added a commit that referenced this pull request Aug 17, 2023
@mp911de
Polishing.
ec7cfa3 
Update since tags. Add missing Override annotation.

See #4070
Original pull request: #4242
mp911de pushed a commit that referenced this pull request Aug 17, 2023
@christophstrobl @mp911de
Introduce AggregationVariable type.
7517527 
This commit introduces a new AggregationVariable type that is intended to better identify variables within a pipeline to avoid mapping failures caused by invalid field names.

Closes #4070
Original pull request: #4242
mp911de added a commit that referenced this pull request Aug 17, 2023
@mp911de
Polishing.
9a68ee4 
Update since tags. Add missing Override annotation.

See #4070
Original pull request: #4242
@mp911de mp911de deleted the issue/4070 branch August 17, 2023 13:01
@mp911de mp911de added this to the 4.1.3 (2023.0.3) milestone Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mp911de mp911de

Labels
type: bug A general bug
Projects
None yet
Milestone
4.1.3 (2023.0.3)
Development

Successfully merging this pull request may close these issues.

Reduce aggregation operation doesn't allow using Variable.VALUE and Variable.THIS with strict mapping
2 participants
@christophstrobl @mp911de