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.

Sign up for GitHub

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

Jump to bottom

Dt on by default #486

Merged
merged 38 commits into from
Oct 18, 2021
Merged

Dt on by default #486

merged 38 commits into from
Oct 18, 2021

Conversation

jasonjkeller
Copy link
Contributor

WIP

jasonjkeller and others added 30 commits September 15, 2021 10:01
@jasonjkeller
Enable DT by default
c450c9f
@XiXiaPdx
change default max spans and make it configurable
4b6507d
@XiXiaPdx
remove unused constants
dd1ff87
@XiXiaPdx
add tests
d960ee4
@XiXiaPdx
remove server side test
c9c7be2
@XiXiaPdx
update span config with server side prop
ef93c6f
@XiXiaPdx
a bit of clean up
46ae714
@XiXiaPdx
refactor type and method name
fd0e619
@XiXiaPdx
send span limit metric
4c497a1
@XiXiaPdx
add and use SpanHarvestConfig to set limits
d101ced
@XiXiaPdx
add test to confirm span config max samples updated by server prop
734821e
@XiXiaPdx
remove not thrown exception in signatures and simplify
482e86e
@jasonjkeller
Clean up ExternalAsyncTest
27f37a4
@jasonjkeller
Fix ExternalAsyncTest by disabling DT and enabling CAT
8d13552
@jasonjkeller
Fix APISupportabilityTest by disabling DT and enabling CAT
af16ff2
@jasonjkeller
Fix APITest by disabling DT and enabling CAT
041e981
@jasonjkeller
Fix CrossProcessStateCatApiTest by disabling DT and enabling CAT
7efb3af
@jasonjkeller
Fix InboundHeaderStateTest by disabling DT and enabling CAT
2a50943
@jasonjkeller
Fix BrowserConfigTest by disabling DT and enabling CAT
be7a61c
@jasonjkeller
Fix ErrorServiceTest by setting correct intrinsics asserts when DT is…
d928ee5 
… enabled
@jasonjkeller
Fix TransactionTest by setting correct intrinsics when DT is enabled
7c43976
@jasonjkeller
Fix TransactionEventTest by asserting that CAT attributes aren't pres…
eb0c0d4 
…ent when DT is enabled
@jasonjkeller
Remove comment left by mistake
d6753cf
@jasonjkeller
Fix TransactionTraceTest by disabling DT and enabling CAT
3866aa6
@jasonjkeller
Fix AbstractPriorityTransactionNamingPolicyTest by disabling DT and e…
17522ca 
…nabling CAT
@jasonjkeller
Fix HigherPriorityTransactionNamingPolicyTest by disabling DT and ena…
50a8650 
…bling CAT
@jasonjkeller
Fix SameOrHigherPriorityTransactionNamingPolicyTest by disabling DT a…
4cb11bf 
…nd enabling CAT
@jasonjkeller
Trigger tests
ee3e25b
@XiXiaPdx
Merge pull request #439 from newrelic/enable-dt-by-default
c640840 
Enable DT by default
@XiXiaPdx
Merge branch 'main' of https://github.com/newrelic/newrelic-java-agent …
8d6d510 
…into dt-on-by-default
@jasonjkeller jasonjkeller reopened this Oct 15, 2021
XiXiaPdx
XiXiaPdx approved these changes Oct 18, 2021
View reviewed changes
Copy link
Contributor

@XiXiaPdx XiXiaPdx left a comment

Choose a reason for hiding this comment

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

🎠 🎠 🎠

@jasonjkeller jasonjkeller merged commit 9c7d192 into main Oct 18, 2021
@jasonjkeller jasonjkeller deleted the dt-on-by-default branch October 18, 2021 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@XiXiaPdx XiXiaPdx XiXiaPdx approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Java Engineering Board
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jasonjkeller @XiXiaPdx @kford-newrelic @meiao