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

[Telemetry Query Language] Updated function signal expectations #5408

Merged
merged 5 commits into from
May 25, 2022

Conversation

TylerHelmuth
Copy link
Member

Description:
Removes the python-like syntax examples. Changes language to be clear functions only work one one signal at a time.

Related to #5395

@TylerHelmuth TylerHelmuth requested review from a team and mx-psi May 23, 2022 15:27
@TylerHelmuth TylerHelmuth changed the title [Telemetry Query Language] Updated examples [Telemetry Query Language] Updated function signal expectations May 23, 2022
@codecov
Copy link

codecov bot commented May 23, 2022

Codecov Report

Merging #5408 (b461fe6) into main (37397ab) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #5408      +/-   ##
==========================================
- Coverage   91.00%   90.98%   -0.02%     
==========================================
  Files         190      191       +1     
  Lines       11428    11429       +1     
==========================================
- Hits        10400    10399       -1     
- Misses        809      811       +2     
  Partials      219      219              
Impacted Files Coverage Δ
config/configmap.go 90.09% <0.00%> (ø)
config/mapconverter/expandmapconverter/expand.go 100.00% <0.00%> (ø)
...rter/overwritepropertiesmapconverter/properties.go 100.00% <0.00%> (ø)
config/mapconverter.go 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37397ab...b461fe6. Read the comment docs.

@TylerHelmuth
Copy link
Member Author

/cc @anuraaga

docs/processing.md Show resolved Hide resolved
docs/processing.md Show resolved Hide resolved
@bogdandrutu bogdandrutu added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 25, 2022
@bogdandrutu bogdandrutu merged commit 8dcdb31 into open-telemetry:main May 25, 2022
@TylerHelmuth TylerHelmuth deleted the remove-python-syntax branch May 25, 2022 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants