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

Support isEmptyValue flag for metrics query. #10616

Merged
merged 5 commits into from
Mar 30, 2023

Conversation

wankai123
Copy link
Member

And Fix NPE when query the not exist series indexes in ElasticSearch storage.

  • 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. Closes [Feature] Support isEmptyValue flag for metrics query #10501.

  • Update the CHANGES log.

…y zero or no data.

Fix `NPE` when query the not exist series indexes in ElasticSearch storage
@wankai123 wankai123 added bug Something isn't working and you are sure it's a bug! feature New feature labels Mar 30, 2023
@wankai123 wankai123 added this to the 9.5.0 milestone Mar 30, 2023
wu-sheng
wu-sheng previously approved these changes Mar 30, 2023
Copy link
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

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

LGTM

@wu-sheng wu-sheng merged commit ec2ca35 into apache:master Mar 30, 2023
@wankai123 wankai123 deleted the empty-value branch March 30, 2023 06:52
kezhenxu94 added a commit to kezhenxu94/skywalking that referenced this pull request Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working and you are sure it's a bug! feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Support isEmptyValue flag for metrics query
3 participants