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

Portable Dynamic Destinations #31807

Closed
wants to merge 10 commits into from

Conversation

ahmedabu98
Copy link
Contributor

@ahmedabu98 ahmedabu98 commented Jul 8, 2024

Dynamic destinations for a portable context, where providing a function is not possible. Following the first proposal outlined in the following design doc: https://s.apache.org/portable-dynamic-destinations

The desired experience looks like this:

Beam Row include_exclude (1)

@github-actions github-actions bot added the java label Jul 8, 2024
@github-actions github-actions bot added the io label Aug 28, 2024
@github-actions github-actions bot added the build label Aug 28, 2024
Copy link

codecov bot commented Aug 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.89%. Comparing base (9c0a950) to head (991ef03).
Report is 21 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #31807      +/-   ##
============================================
+ Coverage     58.85%   58.89%   +0.04%     
- Complexity     3032     3070      +38     
============================================
  Files          1123     1126       +3     
  Lines        173493   173626     +133     
  Branches       3306     3312       +6     
============================================
+ Hits         102105   102263     +158     
+ Misses        68051    68021      -30     
- Partials       3337     3342       +5     
Flag Coverage Δ
java 70.03% <ø> (+0.22%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant