From 58c778f343c3b493eed256c47a3b640c419161dd Mon Sep 17 00:00:00 2001 From: Matt Date: Mon, 19 May 2025 16:54:53 -0400 Subject: [PATCH 1/9] Enable use of zstd (using default level) for profile upload compression --- .../profiling-uploader/build.gradle | 1 + .../uploader/CompressingRequestBody.java | 42 +++++++++++++++---- .../profiling/uploader/CompressionType.java | 10 +++-- .../trace/api/config/ProfilingConfig.java | 9 +++- 4 files changed, 51 insertions(+), 11 deletions(-) diff --git a/dd-java-agent/agent-profiling/profiling-uploader/build.gradle b/dd-java-agent/agent-profiling/profiling-uploader/build.gradle index 85f45e93e82..3dcc82bfcb1 100644 --- a/dd-java-agent/agent-profiling/profiling-uploader/build.gradle +++ b/dd-java-agent/agent-profiling/profiling-uploader/build.gradle @@ -31,6 +31,7 @@ dependencies { implementation libs.okhttp implementation libs.lz4 + implementation group: 'io.airlift', name: 'aircompressor', version: '0.27' testImplementation libs.bundles.junit5 testImplementation project(':dd-java-agent:agent-profiling:profiling-testing') diff --git a/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressingRequestBody.java b/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressingRequestBody.java index d41511d1b0a..3e583640503 100644 --- a/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressingRequestBody.java +++ b/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressingRequestBody.java @@ -2,6 +2,7 @@ import datadog.trace.api.Platform; import datadog.trace.api.profiling.RecordingInputStream; +import io.airlift.compress.zstd.ZstdOutputStream; import java.io.BufferedOutputStream; import java.io.IOException; import java.io.InputStream; @@ -86,6 +87,7 @@ interface RetryBackoff { private static final int[] LZ4_MAGIC = new int[] {0x04, 0x22, 0x4D, 0x18}; private static final int ZIP_MAGIC[] = new int[] {80, 75, 3, 4}; private static final int GZ_MAGIC[] = new int[] {31, 139}; + private static final int ZSTD_MAGIC[] = new int[] {0x28, 0xB5, 0x2F, 0xFD}; private final InputStreamSupplier inputStreamSupplier; private final OutputStreamMappingFunction outputStreamMapper; @@ -269,7 +271,24 @@ public void close() throws IOException { */ static boolean isCompressed(@Nonnull final InputStream is) throws IOException { checkMarkSupported(is); - return isGzip(is) || isLz4(is) || isZip(is); + return isGzip(is) || isLz4(is) || isZip(is) || isZstd(is); + } + + /** + * Check whether the stream represents ZSTD data + * + * @param is input stream; must support {@linkplain InputStream#mark(int)} + * @return {@literal true} if the stream represents ZSTD data + * @throws IOException + */ + static boolean isZstd(@Nonnull final InputStream is) throws IOException { + checkMarkSupported(is); + is.mark(ZSTD_MAGIC.length); + try { + return hasMagic(is, ZSTD_MAGIC); + } finally { + is.reset(); + } } /** @@ -331,12 +350,11 @@ private static void checkMarkSupported(@Nonnull final InputStream is) throws IOE private static OutputStreamMappingFunction getOutputStreamMapper( @Nonnull CompressionType compressionType) { - // currently only gzip and off are supported - // this needs to be updated once more compression types are added - compressionType = - (Platform.isNativeImage() && compressionType != CompressionType.OFF - ? CompressionType.GZIP - : compressionType); + // Handle native image compatibility + if (Platform.isNativeImage() && compressionType != CompressionType.OFF) { + compressionType = CompressionType.GZIP; + } + switch (compressionType) { case GZIP: { @@ -346,6 +364,10 @@ private static OutputStreamMappingFunction getOutputStreamMapper( { return out -> out; } + case ZSTD: + { + return CompressingRequestBody::toZstdStream; + } case ON: case LZ4: default: @@ -355,6 +377,12 @@ private static OutputStreamMappingFunction getOutputStreamMapper( } } + private static OutputStream toZstdStream(@Nonnull OutputStream os) throws IOException { + // Default compression level is 3 which provides a good balance between performance and + // compression ratio + return new ZstdOutputStream(os); + } + private static OutputStream toLz4Stream(@Nonnull OutputStream os) throws IOException { return new LZ4FrameOutputStream( os, diff --git a/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java b/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java index 1b8c1a72d0f..648a02ee473 100644 --- a/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java +++ b/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java @@ -12,7 +12,9 @@ enum CompressionType { /** Lower compression ratio with less CPU overhead * */ LZ4, /** Better compression ratio for the price of higher CPU usage * */ - GZIP; + GZIP, + /** High compression ratio with reasonable CPU usage * */ + ZSTD; private static final Logger log = LoggerFactory.getLogger(CompressionType.class); @@ -30,9 +32,11 @@ static CompressionType of(String type) { return LZ4; case "gzip": return GZIP; + case "zstd": + return ZSTD; default: - log.warn("Unrecognizable compression type: {}. Defaulting to 'on'.", type); - return ON; + log.warn("Unrecognizable compression type: {}. Defaulting to 'zstd'.", type); + return ZSTD; } } } diff --git a/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java b/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java index 451542b9daa..7ee07b41dc6 100644 --- a/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java +++ b/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java @@ -34,7 +34,14 @@ public final class ProfilingConfig { public static final String PROFILING_UPLOAD_TIMEOUT = "profiling.upload.timeout"; public static final int PROFILING_UPLOAD_TIMEOUT_DEFAULT = 30; public static final String PROFILING_UPLOAD_COMPRESSION = "profiling.upload.compression"; - public static final String PROFILING_UPLOAD_COMPRESSION_DEFAULT = "on"; + /** + * Default compression value. Supported values are: - "on": equivalent to "zstd" - "off": disables + * compression - "lz4": uses LZ4 compression (fast with moderate compression ratio) - "gzip": uses + * GZIP compression (higher compression ratio but slower) - "zstd": uses ZSTD compression (high + * compression ratio with reasonable performance) + */ + public static final String PROFILING_UPLOAD_COMPRESSION_DEFAULT = "zstd"; + public static final String PROFILING_PROXY_HOST = "profiling.proxy.host"; public static final String PROFILING_PROXY_PORT = "profiling.proxy.port"; public static final int PROFILING_PROXY_PORT_DEFAULT = 8080; From 83e210d88a918e063f6e881e4d4d8fa08642c169 Mon Sep 17 00:00:00 2001 From: Matt Date: Wed, 21 May 2025 10:31:39 -0400 Subject: [PATCH 2/9] Deprecate previous profiling compression flag --- .../controller/ProfilerSettingsSupport.java | 7 +++++-- .../datadog/trace/api/config/ProfilingConfig.java | 13 +++++++++++++ 2 files changed, 18 insertions(+), 2 deletions(-) diff --git a/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java b/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java index 8afbc31889f..04ee13423a9 100644 --- a/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java +++ b/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java @@ -123,10 +123,13 @@ protected ProfilerSettingsSupport( configProvider.getInteger( ProfilingConfig.PROFILING_UPLOAD_TIMEOUT, ProfilingConfig.PROFILING_UPLOAD_TIMEOUT_DEFAULT); + // First try the new debug upload compression property, and fall back to the deprecated one uploadCompression = configProvider.getString( - ProfilingConfig.PROFILING_UPLOAD_COMPRESSION, - ProfilingConfig.PROFILING_UPLOAD_COMPRESSION_DEFAULT); + ProfilingConfig.PROFILING_DEBUG_UPLOAD_COMPRESSION, + configProvider.getString( + ProfilingConfig.PROFILING_UPLOAD_COMPRESSION, + ProfilingConfig.PROFILING_UPLOAD_COMPRESSION_DEFAULT)); allocationProfilingEnabled = configProvider.getBoolean( ProfilingConfig.PROFILING_ALLOCATION_ENABLED, diff --git a/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java b/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java index 7ee07b41dc6..bd777a662cb 100644 --- a/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java +++ b/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java @@ -33,6 +33,11 @@ public final class ProfilingConfig { "profiling.jfr-template-override-file"; public static final String PROFILING_UPLOAD_TIMEOUT = "profiling.upload.timeout"; public static final int PROFILING_UPLOAD_TIMEOUT_DEFAULT = 30; + /** + * @deprecated Use {@link #PROFILING_DEBUG_UPLOAD_COMPRESSION} instead. This will be removed in a + * future release. + */ + @Deprecated public static final String PROFILING_UPLOAD_COMPRESSION = "profiling.upload.compression"; /** * Default compression value. Supported values are: - "on": equivalent to "zstd" - "off": disables @@ -199,6 +204,14 @@ public final class ProfilingConfig { public static final String PROFILING_DEBUG_DUMP_PATH = "profiling.debug.dump_path"; public static final String PROFILING_DEBUG_JFR_DISABLED = "profiling.debug.jfr.disabled"; + /** + * Configuration for profile upload compression. Supported values are: - "on": equivalent to + * "zstd" - "off": disables compression - "lz4": uses LZ4 compression (fast with moderate + * compression ratio) - "gzip": uses GZIP compression (higher compression ratio but slower) - + * "zstd": uses ZSTD compression (high compression ratio with reasonable performance) + */ + public static final String PROFILING_DEBUG_UPLOAD_COMPRESSION = + "profiling.debug.upload.compression"; public static final String PROFILING_CONTEXT_ATTRIBUTES = "profiling.context.attributes"; From b16250dd8f14ff2f43a27a2ad953e08413ed7053 Mon Sep 17 00:00:00 2001 From: Matt Date: Wed, 21 May 2025 13:18:03 -0400 Subject: [PATCH 3/9] Bump max tracer size to 32 mb --- dd-java-agent/build.gradle | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dd-java-agent/build.gradle b/dd-java-agent/build.gradle index 1b2c3804acb..d526ac19068 100644 --- a/dd-java-agent/build.gradle +++ b/dd-java-agent/build.gradle @@ -306,7 +306,7 @@ tasks.register('checkAgentJarSize').configure { doLast { // Arbitrary limit to prevent unintentional increases to the agent jar size // Raise or lower as required - assert shadowJar.archiveFile.get().getAsFile().length() <= 31 * 1024 * 1024 + assert shadowJar.archiveFile.get().getAsFile().length() <= 32 * 1024 * 1024 } dependsOn "shadowJar" From 89f2fb06d746ab4b29e07dbe344cbc696b26e469 Mon Sep 17 00:00:00 2001 From: Matt Date: Wed, 21 May 2025 14:31:56 -0400 Subject: [PATCH 4/9] Update dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java Co-authored-by: Jaroslav Bachorik --- .../src/main/java/datadog/trace/api/config/ProfilingConfig.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java b/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java index bd777a662cb..1c67a41bd89 100644 --- a/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java +++ b/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java @@ -45,7 +45,7 @@ public final class ProfilingConfig { * GZIP compression (higher compression ratio but slower) - "zstd": uses ZSTD compression (high * compression ratio with reasonable performance) */ - public static final String PROFILING_UPLOAD_COMPRESSION_DEFAULT = "zstd"; + public static final String PROFILING_DEBUG_UPLOAD_COMPRESSION_DEFAULT = "zstd"; public static final String PROFILING_PROXY_HOST = "profiling.proxy.host"; public static final String PROFILING_PROXY_PORT = "profiling.proxy.port"; From 6ec49516f3c04906165921150801c2696fe68ecb Mon Sep 17 00:00:00 2001 From: Matt Date: Wed, 21 May 2025 14:35:38 -0400 Subject: [PATCH 5/9] Nits --- .../profiling/controller/ProfilerSettingsSupport.java | 5 ++--- .../agent-profiling/profiling-uploader/build.gradle | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java b/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java index 04ee13423a9..03f279ea690 100644 --- a/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java +++ b/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java @@ -127,9 +127,8 @@ protected ProfilerSettingsSupport( uploadCompression = configProvider.getString( ProfilingConfig.PROFILING_DEBUG_UPLOAD_COMPRESSION, - configProvider.getString( - ProfilingConfig.PROFILING_UPLOAD_COMPRESSION, - ProfilingConfig.PROFILING_UPLOAD_COMPRESSION_DEFAULT)); + ProfilingConfig.PROFILING_UPLOAD_COMPRESSION_DEFAULT, + ProfilingConfig.PROFILING_UPLOAD_COMPRESSION); allocationProfilingEnabled = configProvider.getBoolean( ProfilingConfig.PROFILING_ALLOCATION_ENABLED, diff --git a/dd-java-agent/agent-profiling/profiling-uploader/build.gradle b/dd-java-agent/agent-profiling/profiling-uploader/build.gradle index 3dcc82bfcb1..068451b437b 100644 --- a/dd-java-agent/agent-profiling/profiling-uploader/build.gradle +++ b/dd-java-agent/agent-profiling/profiling-uploader/build.gradle @@ -31,7 +31,7 @@ dependencies { implementation libs.okhttp implementation libs.lz4 - implementation group: 'io.airlift', name: 'aircompressor', version: '0.27' + implementation group: 'io.airlift', name: 'aircompressor', version: '2.0.2' testImplementation libs.bundles.junit5 testImplementation project(':dd-java-agent:agent-profiling:profiling-testing') From 9590171f2a26f7709a3e49a5bfe0001d84761b3f Mon Sep 17 00:00:00 2001 From: Matt Date: Wed, 21 May 2025 14:39:01 -0400 Subject: [PATCH 6/9] Nits --- .../datadog/profiling/controller/ProfilerSettingsSupport.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java b/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java index 03f279ea690..fa45acd9581 100644 --- a/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java +++ b/dd-java-agent/agent-profiling/profiling-controller/src/main/java/com/datadog/profiling/controller/ProfilerSettingsSupport.java @@ -127,7 +127,7 @@ protected ProfilerSettingsSupport( uploadCompression = configProvider.getString( ProfilingConfig.PROFILING_DEBUG_UPLOAD_COMPRESSION, - ProfilingConfig.PROFILING_UPLOAD_COMPRESSION_DEFAULT, + ProfilingConfig.PROFILING_DEBUG_UPLOAD_COMPRESSION_DEFAULT, ProfilingConfig.PROFILING_UPLOAD_COMPRESSION); allocationProfilingEnabled = configProvider.getBoolean( From b48351ec271dda881f3957ef7a43768077faa54b Mon Sep 17 00:00:00 2001 From: Matt Date: Wed, 21 May 2025 15:17:36 -0400 Subject: [PATCH 7/9] Revert defaults briefly for org2 --- .../datadog/profiling/uploader/CompressionType.java | 2 +- .../java/datadog/trace/api/config/ProfilingConfig.java | 10 +++++----- .../src/main/java/datadog/trace/api/Config.java | 2 +- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java b/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java index 648a02ee473..4d11b5dc597 100644 --- a/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java +++ b/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java @@ -36,7 +36,7 @@ static CompressionType of(String type) { return ZSTD; default: log.warn("Unrecognizable compression type: {}. Defaulting to 'zstd'.", type); - return ZSTD; + return ON; } } } diff --git a/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java b/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java index 1c67a41bd89..b0408d9f760 100644 --- a/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java +++ b/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java @@ -40,12 +40,12 @@ public final class ProfilingConfig { @Deprecated public static final String PROFILING_UPLOAD_COMPRESSION = "profiling.upload.compression"; /** - * Default compression value. Supported values are: - "on": equivalent to "zstd" - "off": disables - * compression - "lz4": uses LZ4 compression (fast with moderate compression ratio) - "gzip": uses - * GZIP compression (higher compression ratio but slower) - "zstd": uses ZSTD compression (high - * compression ratio with reasonable performance) + * Default compression value. Supported values are: - "on": equivalent to "lz4", will later be + * "zstd" - "off": disables compression - "lz4": uses LZ4 compression (fast with moderate + * compression ratio) - "gzip": uses GZIP compression (higher compression ratio but slower) - + * "zstd": uses ZSTD compression (high compression ratio with reasonable performance) */ - public static final String PROFILING_DEBUG_UPLOAD_COMPRESSION_DEFAULT = "zstd"; + public static final String PROFILING_DEBUG_UPLOAD_COMPRESSION_DEFAULT = "lz4"; public static final String PROFILING_PROXY_HOST = "profiling.proxy.host"; public static final String PROFILING_PROXY_PORT = "profiling.proxy.port"; diff --git a/internal-api/src/main/java/datadog/trace/api/Config.java b/internal-api/src/main/java/datadog/trace/api/Config.java index 90370b23ace..25ddfd78d68 100644 --- a/internal-api/src/main/java/datadog/trace/api/Config.java +++ b/internal-api/src/main/java/datadog/trace/api/Config.java @@ -1256,7 +1256,7 @@ PROFILING_DATADOG_PROFILER_ENABLED, isDatadogProfilerSafeInCurrentEnvironment()) configProvider.getInteger(PROFILING_UPLOAD_TIMEOUT, PROFILING_UPLOAD_TIMEOUT_DEFAULT); profilingUploadCompression = configProvider.getString( - PROFILING_UPLOAD_COMPRESSION, PROFILING_UPLOAD_COMPRESSION_DEFAULT); + PROFILING_UPLOAD_COMPRESSION, PROFILING_DEBUG_UPLOAD_COMPRESSION_DEFAULT); profilingProxyHost = configProvider.getString(PROFILING_PROXY_HOST); profilingProxyPort = configProvider.getInteger(PROFILING_PROXY_PORT, PROFILING_PROXY_PORT_DEFAULT); From d0e1566694c794a5e62463aa8b5d54350d1c30e7 Mon Sep 17 00:00:00 2001 From: Matt Date: Wed, 21 May 2025 19:22:27 -0400 Subject: [PATCH 8/9] Include ZSTD tests --- .../uploader/CompressingRequestBodyTest.java | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/dd-java-agent/agent-profiling/profiling-uploader/src/test/java/com/datadog/profiling/uploader/CompressingRequestBodyTest.java b/dd-java-agent/agent-profiling/profiling-uploader/src/test/java/com/datadog/profiling/uploader/CompressingRequestBodyTest.java index 9dd3a6e23ac..adcf92de8e1 100644 --- a/dd-java-agent/agent-profiling/profiling-uploader/src/test/java/com/datadog/profiling/uploader/CompressingRequestBodyTest.java +++ b/dd-java-agent/agent-profiling/profiling-uploader/src/test/java/com/datadog/profiling/uploader/CompressingRequestBodyTest.java @@ -6,6 +6,8 @@ import static org.mockito.Mockito.when; import datadog.trace.api.profiling.RecordingInputStream; +import io.airlift.compress.zstd.ZstdInputStream; +import io.airlift.compress.zstd.ZstdOutputStream; import java.io.BufferedInputStream; import java.io.ByteArrayInputStream; import java.io.ByteArrayOutputStream; @@ -186,6 +188,15 @@ void writeTo(CompressionType compressionType) throws IOException { assertEquals(compressed.length, instance.getWrittenBytes()); break; } + case ZSTD: + { + assertTrue(CompressingRequestBody.isZstd(compressedStream)); + byte[] uncompressed = IOUtils.toByteArray(new ZstdInputStream(compressedStream)); + assertArrayEquals(recordingData, uncompressed); + assertEquals(recordingData.length, instance.getReadBytes()); + assertEquals(compressed.length, instance.getWrittenBytes()); + break; + } } } @@ -210,6 +221,11 @@ void writeToRecompression(CompressionType targetType) throws IOException { compressedStream = new GZIPOutputStream(baos); break; } + case ZSTD: + { + compressedStream = new ZstdOutputStream(baos); + break; + } } assertNotNull(compressedStream); From 6e76ffb3deb11e6a2357514d9355a3f9615b1cb6 Mon Sep 17 00:00:00 2001 From: Matt Date: Thu, 22 May 2025 07:36:35 -0400 Subject: [PATCH 9/9] Straggling comments and log line zstd-reversion --- .../com/datadog/profiling/uploader/CompressionType.java | 2 +- .../java/datadog/trace/api/config/ProfilingConfig.java | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java b/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java index 4d11b5dc597..e86e990ba17 100644 --- a/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java +++ b/dd-java-agent/agent-profiling/profiling-uploader/src/main/java/com/datadog/profiling/uploader/CompressionType.java @@ -35,7 +35,7 @@ static CompressionType of(String type) { case "zstd": return ZSTD; default: - log.warn("Unrecognizable compression type: {}. Defaulting to 'zstd'.", type); + log.warn("Unrecognizable compression type: {}. Defaulting to 'lz4'.", type); return ON; } } diff --git a/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java b/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java index b0408d9f760..14212fe6c43 100644 --- a/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java +++ b/dd-trace-api/src/main/java/datadog/trace/api/config/ProfilingConfig.java @@ -205,10 +205,10 @@ public final class ProfilingConfig { public static final String PROFILING_DEBUG_DUMP_PATH = "profiling.debug.dump_path"; public static final String PROFILING_DEBUG_JFR_DISABLED = "profiling.debug.jfr.disabled"; /** - * Configuration for profile upload compression. Supported values are: - "on": equivalent to - * "zstd" - "off": disables compression - "lz4": uses LZ4 compression (fast with moderate - * compression ratio) - "gzip": uses GZIP compression (higher compression ratio but slower) - - * "zstd": uses ZSTD compression (high compression ratio with reasonable performance) + * Configuration for profile upload compression. Supported values are: - "on": equivalent to "lz4" + * - "off": disables compression - "lz4": uses LZ4 compression (fast with moderate compression + * ratio) - "gzip": uses GZIP compression (higher compression ratio but slower) - "zstd": uses + * ZSTD compression (high compression ratio with reasonable performance) */ public static final String PROFILING_DEBUG_UPLOAD_COMPRESSION = "profiling.debug.upload.compression";