Add support for Azure2021 trace type#748
Draft
16fb wants to merge 7 commits intovhive-serverless:mainfrom
Draft
Conversation
Signed-off-by: Bryan Wong wongwenpingbryan@gmail.com
Added azure2021_trace_parser and azure2021_trace_parser_test (basic). Added path string detection of Azure2021 or Azure2019 path. Signed-off-by: Bryan Wong wongwenpingbryan@gmail.com
Signed-off-by: Bryan Wong wongwenpingbryan@gmail.com
Fix logic bug for iat calculation when slices have 3 or more invocations. Added truncation of invocations if "end_timestamp" exceeds "ExperimentDuration" in config json. Added iat input checks for negative iat values. Added examples test csv(s). Signed-off-by: Bryan Wong wongwenpingbryan@gmail.com
Signed-off-by: Bryan Wong wongwenpingbryan@gmail.com
Signed-off-by: Bryan Wong wongwenpingbryan@gmail.com
Removed redundant parameter for functionSpecifiction() Add TestAzure2021ParserWrapper() Signed-off-by: Bryan Wong wongwenpingbryan@gmail.com
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
A small summary of the requirements (in one/two sentences).
Implementation Notes ⚒️
Created "azure_2021_trace_parser.go" with main parsing logic.
Modified "loader.go" to expect azure2021 trace type.
External Dependencies 🍀
none (N/A)
Breaking API Changes⚠️
none (N/A), Other trace types should* be unaffected.