-
Notifications
You must be signed in to change notification settings - Fork 1
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
Reduce operator 1 #23
base: master
Are you sure you want to change the base?
Conversation
@Neel-Shah-29 : can you please fix these conflict so I can test the Reduce operator ? |
f675d07
to
f04107f
Compare
Done with the changes sir now you can review it once |
@Neel-Shah-29 : can you please fix the conflicts with respect to the new master ? |
f04107f
to
a0f2f3d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have some comments on fixing the generated code.
…hape in the Reduce Operator
Fix also correct namespace for reference output in reduce tests
b32397f
to
75d47d7
Compare
This Pull request:
Changes or fixes:
Checklist:
This PR fixes #