-
Notifications
You must be signed in to change notification settings - Fork 873
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
Feat: velaql support indexing into exported array field #5320
Feat: velaql support indexing into exported array field #5320
Conversation
eb4c63a
to
7f693aa
Compare
Signed-off-by: hnd4r7 <307365651@qq.com>
7f693aa
to
441d8f0
Compare
Codecov ReportBase: 46.37% // Head: 46.21% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #5320 +/- ##
==========================================
- Coverage 46.37% 46.21% -0.17%
==========================================
Files 294 251 -43
Lines 45744 42245 -3499
==========================================
- Hits 21214 19523 -1691
+ Misses 22205 20515 -1690
+ Partials 2325 2207 -118
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Successfully created backport PR #5323 for |
Signed-off-by: hnd4r7 <307365651@qq.com> Signed-off-by: hnd4r7 <307365651@qq.com>
Signed-off-by: hnd4r7 <307365651@qq.com> Signed-off-by: hnd4r7 <307365651@qq.com>
Signed-off-by: hnd4r7 307365651@qq.com
Description of your changes
Support indexing array field in the exported value of the velaql query result to satisfy certain needs such as getting the cluster name of the first pod in component-pod-view.
Before,
![image](https://user-images.githubusercontent.com/10682438/211798505-b5571b33-4239-4780-aefd-fc34145a267b.png)
![image](https://user-images.githubusercontent.com/10682438/211798524-01cb9858-3e52-4869-a682-69e24a1b3c8c.png)
After,
Fixes #
I have:
make reviewable
to ensure this PR is ready for review.backport release-x.y
labels to auto-backport this PR if necessary.How has this code been tested
Special notes for your reviewer