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

build link b/w parent-child #208

Merged
merged 9 commits into from
Oct 8, 2021

Conversation

Sunn-y-Arora
Copy link
Contributor

@Sunn-y-Arora Sunn-y-Arora commented Jun 22, 2021

Description

Adding support for the follow_from construct for building parent-child relationship.
Ref: hypertrace/hypertrace#250

Data-model changes PR hypertrace/data-model#26

@codecov
Copy link

codecov bot commented Jun 22, 2021

Codecov Report

Merging #208 (19db96d) into main (8e72605) will increase coverage by 0.03%.
The diff coverage is 88.88%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #208      +/-   ##
============================================
+ Coverage     80.26%   80.29%   +0.03%     
  Complexity     1166     1166              
============================================
  Files           102      102              
  Lines          4535     4542       +7     
  Branches        421      423       +2     
============================================
+ Hits           3640     3647       +7     
  Misses          696      696              
  Partials        199      199              
Flag Coverage Δ
unit 80.29% <88.88%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...e/viewgenerator/generators/ViewGeneratorState.java 94.11% <88.88%> (+0.93%) ⬆️

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 8e72605...19db96d. Read the comment docs.

Copy link
Contributor

@surajpuvvada surajpuvvada left a comment

Choose a reason for hiding this comment

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

Can you add unit tests ?

@Sunn-y-Arora Sunn-y-Arora marked this pull request as ready for review October 7, 2021 06:55
@Sunn-y-Arora Sunn-y-Arora requested a review from a team October 7, 2021 06:55
@Sunn-y-Arora
Copy link
Contributor Author

@surajpuvvada added 1 test for traceState with FollowFrom link

@Sunn-y-Arora
Copy link
Contributor Author

@kotharironak is PR good to merge now ?

@kotharironak kotharironak merged commit 31cf9fd into hypertrace:main Oct 8, 2021
@github-actions
Copy link

github-actions bot commented Oct 8, 2021

Unit Test Results

  70 files  ±0    70 suites  ±0   51s ⏱️ -1s
380 tests +1  380 ✔️ +1  0 💤 ±0  0 ❌ ±0 

Results for commit 31cf9fd. ± Comparison against base commit 8e72605.

@Sunn-y-Arora Sunn-y-Arora deleted the follow_from_construct branch October 8, 2021 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants