8000 Service naming: split by jee deployment by amarziali · Pull Request #8064 · DataDog/dd-trace-java · GitHub
[go: up one dir, main page]
Skip to content

Service naming: split by jee deployment #8064

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

Merged
merged 14 commits into from
Dec 12, 2024
Merged

Service naming: split by jee deployment #8064

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
cac1c30
Service naming: split by jee deployment
amarziali Dec 2, 2024
47a72fc
Do not use invokedynamic
amarziali Dec 6, 2024
fe24e62
fix tests
amarziali Dec 6, 2024
e9c2536
collapse enter and local
amarziali Dec 9, 2024
a0b2e0e
Adjust service name for non legacy tracing
amarziali Dec 9, 2024
12df126
Add wildfly ejb test
amarziali Dec 9, 2024
5765d30
limit max java verson to 11 for wildfly 15
amarziali Dec 10, 2024
4b43be2
renaming
amarziali Dec 12, 2024
c2cb1c4
refactor according to the review
amarziali Dec 12, 2024
a40b4dc
remove helper
amarziali Dec 12, 2024
1bdfabf
avoid casting
amarziali Dec 12, 2024
20d93e3
review
amarziali Dec 12, 2024
0559bdb
revert lastindexof
amarziali Dec 12, 2024
933c7fa
enforce checks
amarziali Dec 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix tests
  • Loading branch information
@amarziali
amarziali committed Dec 12, 2024
commit fe24e628e36e874a64318a67b39728e50e807c6e
6 changes: 3 additions & 3 deletions dd-trace-core/src/test/groovy/datadog/trace/core/taginterceptor/TagInterceptorTest.groovy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,7 @@ class TagInterceptorTest extends DDCoreSpecification {
.sampler(new AllSampler())
// equivalent to split-by-tags: tag
.tagInterceptor(new TagInterceptor(true, "my-service",
Collections.singleton(tag), new RuleFlags()))
Collections.singleton(tag), new RuleFlags(), false))
.build()
}

Expand Down Expand Up @@ -700,7 +700,7 @@ class TagInterceptorTest extends DDCoreSpecification {
ServiceNameCollector.INSTANCE = extraServiceProvider
final ruleFlags = Mock(RuleFlags)
ruleFlags.isEnabled(_) >> true
final interceptor = new TagInterceptor(true, "my-service", Collections.singleton(DDTags.SERVICE_NAME), ruleFlags)
final interceptor = new TagInterceptor(true, "my-service", Collections.singleton(DDTags.SERVICE_NAME), ruleFlags, false)

when:
interceptor.interceptServiceName(null, Mock(DDSpanContext), "some-service")
Expand All @@ -715,7 +715,7 @@ class TagInterceptorTest extends DDCoreSpecification {
ServiceNameCollector.INSTANCE = extraServiceProvider
final ruleFlags = Mock(RuleFlags)
ruleFlags.isEnabled(_) >> true
final interceptor = new TagInterceptor(true, "my-service", Collections.singleton("servlet.context"), ruleFlags)
final interceptor = new TagInterceptor(true, "my-service", Collections.singleton("servlet.context"), ruleFlags, false)

when:
interceptor.interceptServletContext(Mock(DDSpanContext), value)
Expand Down
0