Skip to content

Enabling optimized map by default #10016

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
dougqh merged 8 commits into from
Jan 12, 2026
+1 −2
Merged

Enabling optimized map by default #10016

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
3a88838
Enabling optimized map by default
dougqh Nov 21, 2025
b36dbbb
Merge branch 'master' into dougqh/optimized-map-on-by-default
dougqh Nov 25, 2025
aece5b1
Merge branch 'master' into dougqh/optimized-map-on-by-default
dougqh Dec 9, 2025
402adb1
Merge branch 'master' into dougqh/optimized-map-on-by-default
dougqh Dec 9, 2025
65d63fe
Merge branch 'master' into dougqh/optimized-map-on-by-default
dougqh Dec 9, 2025
c958667
Merge branch 'master' into dougqh/optimized-map-on-by-default
dougqh Jan 9, 2026
757a4c0
Merge branch 'master' into dougqh/optimized-map-on-by-default
dougqh Jan 9, 2026
1287c94
Merge branch 'master' into dougqh/optimized-map-on-by-default
dougqh Jan 12, 2026
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
3 changes: 1 addition & 2 deletions internal-api/src/main/java/datadog/trace/api/Config.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2952,8 +2952,7 @@ PROFILING_DATADOG_PROFILER_ENABLED, isDatadogProfilerSafeInCurrentEnvironment())

this.jdkSocketEnabled = configProvider.getBoolean(JDK_SOCKET_ENABLED, true);

this.optimizedMapEnabled =
configProvider.getBoolean(GeneralConfig.OPTIMIZED_MAP_ENABLED, false);
this.optimizedMapEnabled = configProvider.getBoolean(GeneralConfig.OPTIMIZED_MAP_ENABLED, true);
Copy link
Contributor

@AlexeyKuznetsov-DD AlexeyKuznetsov-DD Dec 15, 2025

Choose a reason for hiding this comment

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

Just curious, if it is a config change, do we need to update some other place too?
@mhlidd Could you check this PR too?

Copy link
Contributor Author

@dougqh dougqh Jan 9, 2026

Choose a reason for hiding this comment

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

I think that only applies if we introduce a config variable. This config has been in place for some time now.

this.spanBuilderReuseEnabled =
configProvider.getBoolean(GeneralConfig.SPAN_BUILDER_REUSE_ENABLED, true);
this.tagNameUtf8CacheSize =
Expand Down