[ https://issues.apache.org/jira/browse/CASSANDRA-16575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17317806#comment-17317806 ]
Jacek Lewandowski commented on CASSANDRA-16575: ----------------------------------------------- I was wrong, the compression tests are aimed to test sstable compression which is explicitly disabled when running just {{unit}} tests. SchemaLoader forces no compression for each table that is created through that utility in tests; Also when compression is enabled, it forces snappy compressor to be used for those tables. All other tables are created with the default LZ4. Therefore it seems like the compression configuration has nothing to do with commit log compression and the only thing we should consider to use the current default compressor LZ4 instead of Snappy and remove the misleading fragment of the build. I'm still thinking that we should allow for passing test configuration without the need to modify the build. > "compression" configuration in unit tests is incorrect and actually redundant > ----------------------------------------------------------------------------- > > Key: CASSANDRA-16575 > URL: https://issues.apache.org/jira/browse/CASSANDRA-16575 > Project: Cassandra > Issue Type: Bug > Components: Test/unit > Reporter: Jacek Lewandowski > Priority: Normal > > When running unit tests with "compression" configuration, we actually not use > compression at all we run just unit tests without compression (which is > redundant when running on CI). > The reason is that for the compression configuration we (try to) do the > following: > {code} > <property name="compressed_yaml" > value="${build.test.dir}/cassandra.compressed.yaml"/> > <concat destfile="${compressed_yaml}"> > <fileset file="${test.conf}/cassandra.yaml"/> > <fileset file="${test.conf}/commitlog_compression.yaml"/> > </concat> > {code} > however, {{cassandra.compressed.yaml}} has been replaced in CASSANDRA-14482 > by two files {{commitlog_compression_LZ4.yaml}} and > {{commitlog_compression_Zstd.yaml}} without changing build configuration. > All in all we could fix that but I doubt it makes any sense since {{cdc}} > configuration already uses compressed commit log anyway so it feels redundant > to test compression alone for each CI run. > I vote for removing compression configuration and actually allow to manually > specify yaml file to be used for testing as a test property, as well as allow > to manually specify extra system properties to be passed to the test. That > would make the build more flexible and remove a bit of redundant code. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org