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

Adding support of multiple operators in one fragment #660

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

lipoja
Copy link
Contributor

@lipoja lipoja commented Apr 8, 2024

Adding support of multiple operators in one FBC fragment.

[CLOUDDST-22242]

JAVGan
JAVGan previously approved these changes Apr 10, 2024
Copy link
Contributor

@chandwanitulsi chandwanitulsi left a comment

Choose a reason for hiding this comment

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

Added a few suggestions, overall LGTM

@lipoja lipoja dismissed stale reviews from yashvardhannanavati and JAVGan via d3d88a3 April 15, 2024 06:10
@lipoja lipoja force-pushed the fbc_multiple_operators branch 2 times, most recently from d3d88a3 to 5fbb253 Compare April 15, 2024 06:12
@lipoja
Copy link
Contributor Author

lipoja commented Apr 15, 2024

Rebased and added @chandwanitulsi suggestions.

@lipoja lipoja force-pushed the fbc_multiple_operators branch 2 times, most recently from 499357d to f639d48 Compare April 16, 2024 14:14
@lipoja lipoja merged commit a2869aa into release-engineering:master Apr 17, 2024
2 checks passed
@lipoja lipoja deleted the fbc_multiple_operators branch April 17, 2024 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants