Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

add aggregate operation #27

Merged
merged 3 commits into from
May 16, 2023
Merged

add aggregate operation #27

merged 3 commits into from
May 16, 2023

Conversation

preyunk
Copy link
Contributor

@preyunk preyunk commented May 16, 2023

Add support for aggregate operations on queryset,
user can pass multiple aggregate operations, also all operations are stopped after aggregate.

Added test cases

Add support for aggregate operations on queryset,
user can pass multiple aggregate operations, also all operations
are stopped after aggregate.

Added test cases

Signed-off-by: Priyank Singh <[email protected]>
@preyunk preyunk requested a review from piyusql May 16, 2023 07:40
@vmwclabot
Copy link

@preyunk, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <[email protected]> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

@vmwclabot
Copy link

@preyunk, you must sign every commit in this pull request acknowledging our Developer Certificate of Origin before your changes are merged. This can be done by adding Signed-off-by: John Doe <[email protected]> to the last line of each Git commit message. The e-mail address used to sign must match the e-mail address of the Git author. Click here to view the Developer Certificate of Origin agreement.

Copy link
Contributor

@piyusql piyusql left a comment

Choose a reason for hiding this comment

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

Looks good

@piyusql piyusql merged commit 86b269b into main May 16, 2023
@piyusql piyusql deleted the aggregate-opt branch May 24, 2023 14:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants