-
Notifications
You must be signed in to change notification settings - Fork 82
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
Add mod revision check to write requests(measure/stream) #322
Conversation
b5a4104
to
935b111
Compare
d2a163e
to
12fd875
Compare
Codecov Report
@@ Coverage Diff @@
## main #322 +/- ##
==========================================
- Coverage 40.63% 40.59% -0.05%
==========================================
Files 110 110
Lines 11726 11775 +49
==========================================
+ Hits 4765 4780 +15
- Misses 6493 6526 +33
- Partials 468 469 +1
... and 2 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@@ -33,7 +33,7 @@ services: | |||
- sw_agent:/skywalking-java-agent | |||
|
|||
oap: | |||
image: "ghcr.io/apache/skywalking/oap:${SW_OAP_COMMIT}" | |||
image: "ghcr.io/apache/skywalking-banyandb-java-client/oap:${SW_OAP_COMMIT}" |
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.
Use this image for integration testing
show case 2023-09-11+00.47.28_out.mov |
654ea1c
to
80ef1d3
Compare
pkg/query/logical/common.go
Outdated
"the number of tags %d in %s is less then expected %d", | ||
len(parsedTagFamily.Tags), familyName, len(refs)) | ||
} | ||
tags := make([]*modelv1.Tag, len(parsedTagFamily.GetTags())) |
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.
Fix query failure caused by schema changes
1a6baeb
to
8d55363
Compare
- Support for create/update schema check mod revision - Support for write data check mod revision - Adapt to web console requests - Update OAP e2e image - Fix query failure caused by schema change
8d55363
to
601615d
Compare
PTAL |
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.
If it's UI related, attach the screenshots below.
If this pull request closes/resolves/fixes an existing issue, replace the issue number. Close Add mod revision check to write requests skywalking#10956.
Update the
CHANGES
log.