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.

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

Implement error handling that will filter the rows that have any error #93

Closed
Tracked by #83
benedeki opened this issue Mar 15, 2023 · 0 comments
Closed
Tracked by #83
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@benedeki
Copy link
Contributor

benedeki commented Mar 15, 2023

Background

One of the provided ErrorHandling implementations. The ErrorMessage marks rows that are to be removed from the result and on aggregation actually filtered out.

Feature

Don't write errors anywhere, aggregation filters out rows that had some error.

Proposed Solution

Solution Ideas:

  1. The ErrorColumn is a boolean column
  2. When coalesce on aggregation of the ErrorColumns is true filter out the row
@benedeki benedeki added enhancement New feature or request good first issue Good for newcomers labels Mar 15, 2023
@TebaleloS TebaleloS self-assigned this Mar 28, 2023
TebaleloS added a commit that referenced this issue Apr 3, 2023
TebaleloS added a commit that referenced this issue Apr 3, 2023
TebaleloS added a commit that referenced this issue Apr 3, 2023
TebaleloS added a commit that referenced this issue Apr 18, 2023
TebaleloS added a commit that referenced this issue Apr 18, 2023
TebaleloS added a commit that referenced this issue Apr 19, 2023
TebaleloS added a commit that referenced this issue Apr 19, 2023
TebaleloS added a commit that referenced this issue Apr 19, 2023
TebaleloS added a commit that referenced this issue Apr 19, 2023
TebaleloS added a commit that referenced this issue Apr 19, 2023
TebaleloS added a commit that referenced this issue Apr 20, 2023
TebaleloS added a commit that referenced this issue Apr 25, 2023
@benedeki benedeki moved this from 🆕 To groom to 🏗 In progress in CPS small repos project Apr 28, 2023
TebaleloS added a commit that referenced this issue Apr 28, 2023
TebaleloS added a commit that referenced this issue Apr 28, 2023
TebaleloS added a commit that referenced this issue May 2, 2023
TebaleloS added a commit that referenced this issue May 2, 2023
TebaleloS added a commit that referenced this issue May 2, 2023
TebaleloS added a commit that referenced this issue May 4, 2023
TebaleloS added a commit that referenced this issue May 4, 2023
TebaleloS added a commit that referenced this issue May 5, 2023
TebaleloS added a commit that referenced this issue May 5, 2023
TebaleloS added a commit that referenced this issue May 5, 2023
TebaleloS added a commit that referenced this issue May 8, 2023
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in CPS small repos project May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
Archived in project
Development

No branches or pull requests

2 participants