Skip to content

Commit 43daa46

Browse files
committed
Enable LZ4 for page exchange
1 parent 0314439 commit 43daa46

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

core/trino-main/src/main/java/io/trino/FeaturesConfig.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ public class FeaturesConfig
9393
/**
9494
* default value is overwritten for fault tolerant execution in {@link #applyFaultTolerantExecutionDefaults()}}
9595
*/
96-
private CompressionCodec exchangeCompressionCodec = NONE;
96+
private CompressionCodec exchangeCompressionCodec = LZ4;
9797
private boolean pagesIndexEagerCompactionEnabled;
9898
private boolean omitDateTimeTypePrecision;
9999
private int maxRecursionDepth = 10;

core/trino-main/src/main/java/io/trino/client/direct/DirectTrinoClient.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
import static io.trino.SystemSessionProperties.getRetryPolicy;
4747
import static io.trino.execution.QueryState.FAILED;
4848
import static io.trino.execution.QueryState.FINISHING;
49-
import static io.trino.execution.buffer.CompressionCodec.NONE;
49+
import static io.trino.execution.buffer.CompressionCodec.LZ4;
5050
import static io.trino.memory.context.AggregatedMemoryContext.newSimpleAggregatedMemoryContext;
5151
import static io.trino.spi.StandardErrorCode.GENERIC_INTERNAL_ERROR;
5252
import static java.util.Objects.requireNonNull;
@@ -94,7 +94,7 @@ public DispatchQuery execute(SessionContext sessionContext, @Language("SQL") Str
9494
}
9595
});
9696

97-
PageDeserializer pageDeserializer = new PagesSerdeFactory(blockEncodingSerde, NONE).createDeserializer(Optional.empty());
97+
PageDeserializer pageDeserializer = new PagesSerdeFactory(blockEncodingSerde, LZ4).createDeserializer(Optional.empty());
9898
for (QueryState state = queryManager.getQueryState(queryId);
9999
(state != FAILED) &&
100100
!exchangeClient.isFinished() &&

core/trino-main/src/test/java/io/trino/sql/analyzer/TestFeaturesConfig.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
import static io.airlift.units.DataSize.Unit.GIGABYTE;
2929
import static io.airlift.units.DataSize.Unit.KILOBYTE;
3030
import static io.airlift.units.DataSize.Unit.MEGABYTE;
31-
import static io.trino.execution.buffer.CompressionCodec.NONE;
31+
import static io.trino.execution.buffer.CompressionCodec.LZ4;
3232
import static io.trino.execution.buffer.CompressionCodec.ZSTD;
3333
import static io.trino.sql.analyzer.RegexLibrary.JONI;
3434
import static io.trino.sql.analyzer.RegexLibrary.RE2J;
@@ -53,7 +53,7 @@ public void testDefaults()
5353
.setSpillMaxUsedSpaceThreshold(0.9)
5454
.setMemoryRevokingThreshold(0.9)
5555
.setMemoryRevokingTarget(0.5)
56-
.setExchangeCompressionCodec(NONE)
56+
.setExchangeCompressionCodec(LZ4)
5757
.setExchangeDataIntegrityVerification(DataIntegrityVerification.ABORT)
5858
.setPagesIndexEagerCompactionEnabled(false)
5959
.setFilterAndProjectMinOutputPageSize(DataSize.of(500, KILOBYTE))

0 commit comments

Comments
 (0)