Skip to content

Conversation

@jketema
Copy link
Contributor

@jketema jketema commented Apr 24, 2025

The context for this is that we might want to raise it to 3 for C++ in combination with some other changes.

@jketema jketema changed the title Dataflow: Makde default field flow branch limit configurable per language Dataflow: Make default field flow branch limit configurable per language Apr 24, 2025
Copy link
Contributor

@aschackmull aschackmull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jketema jketema marked this pull request as ready for review April 24, 2025 14:44
@jketema jketema added the no-change-note-required This PR does not need a change note label Apr 24, 2025
@jketema
Copy link
Contributor Author

jketema commented Apr 24, 2025

DCA experiments all look ok.

@jketema jketema merged commit 21c9708 into github:main Apr 24, 2025
36 of 37 checks passed
@jketema jketema deleted the fieldflow branch April 24, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DataFlow Library no-change-note-required This PR does not need a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants