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.

Sign up for GitHub

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

Jump to bottom

[ospp] Supports measure aggregate function avg and min, and test cases. #521

Merged
merged 22 commits into from
Sep 4, 2024
Merged

[ospp] Supports measure aggregate function avg and min, and test cases. #521

merged 22 commits into from
Sep 4, 2024

Conversation

StLeoX
Copy link
Contributor

@StLeoX StLeoX commented Aug 25, 2024

Supports measure aggregate function (MAF)

  • If this is non-trivial feature, paste the links/URLs to the design doc.
  • Update the documentation to include this new feature.
  • Tests(including UT, IT, E2E) are added to verify the new feature.

Currently avg and min.

StLeoX and others added 17 commits June 28, 2024 16:01
@StLeoX
[API] MeasureAggregateFunctionService Support
00b2bdb
@StLeoX
fix check
38aec7f
@wu-sheng
Merge branch 'main' into dev_measure_aggregate_function
4c94a7c
@StLeoX
use MeasureAggregate enum
ad3b14a
@StLeoX
Merge branch 'main' into dev_measure_aggregate_function
d456b4f
@hanahmily
Merge branch 'main' into dev_measure_aggregate_function
acfe475
@StLeoX
add PERCENTILE enum for MEASURE_AGGREGATE
e05a97b
@StLeoX
add comments for MeasureAggregate enum
e1b7e98
@StLeoX
Merge branch 'main' into dev_measure_aggregate_function
b94f21b
@StLeoX
fix enum name error in FieldSpec
7c9d27f
@StLeoX
a
11ccee1
@StLeoX
fix wrong enum name in FieldSpec
cfc3811
@StLeoX
Supports measure aggregate function avg and min, and test cases.
31763db
@StLeoX
Merge remote-tracking branch 'origin/dev_measure_aggregate_function' …
13c1180 
…into dev_measure_aggregate_function
@StLeoX
Revert "a"
98076ff 
This reverts commit 11ccee1.
@StLeoX
Merge branch 'main' into dev_measure_aggregate_function
515b3f9 
# Conflicts:
#	api/proto/banyandb/database/v1/schema.proto
#	docs/api-reference.md
@StLeoX
fix Merge error
deefc6e
@wu-sheng wu-sheng added the enhancement New feature or request label Aug 25, 2024
hanahmily
hanahmily requested changes Aug 29, 2024
View reviewed changes
banyand/measure/aggregate/aggregate_function.go Show resolved Hide resolved
banyand/measure/aggregate/aggregate_function.go Outdated Show resolved Hide resolved
banyand/measure/aggregate/aggregate_function.go Outdated Show resolved Hide resolved
banyand/measure/aggregate/aggregate_function.go Show resolved Hide resolved
banyand/measure/aggregate/aggregate_function.go Outdated Show resolved Hide resolved
banyand/measure/aggregate/aggregate_function.go Outdated Show resolved Hide resolved
banyand/measure/aggregate/aggregate_function.go Outdated Show resolved Hide resolved
banyand/measure/aggregate/avg.go Outdated Show resolved Hide resolved
banyand/measure/aggregate/avg_test.go Outdated Show resolved Hide resolved
@StLeoX StLeoX requested a review from hanahmily August 30, 2024 03:47
@hanahmily hanahmily merged commit 17dc8c0 into apache:main Sep 4, 2024
15 checks passed
@wu-sheng wu-sheng added this to the 0.7.0 milestone Sep 4, 2024
StLeoX added a commit to StLeoX/skywalking-banyandb that referenced this pull request Sep 24, 2024
@StLeoX @wu-sheng @hanahmily
[ospp] Supports measure aggregate function avg and min, and test case…
310e867 
…s. (apache#521)

* [API] MeasureAggregateFunctionService Support

* Supports measure aggregate function avg and min.

---------

Co-authored-by: 吴晟 Wu Sheng <[email protected]>
Co-authored-by: Gao Hongtao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hanahmily hanahmily hanahmily approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
0.7.0
Development

Successfully merging this pull request may close these issues.
3 participants
@StLeoX @hanahmily @wu-sheng