Skip to content

Commit 17ec517

Browse files
authored
feat(clp-s): Make lossless float retention the default behaviour for clp-s. (#1321)
1 parent 0cd61db commit 17ec517

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

components/core/src/clp_s/CommandLineArguments.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -245,9 +245,9 @@ CommandLineArguments::parse_arguments(int argc, char const** argv) {
245245
po::bool_switch(&m_print_archive_stats),
246246
"Print statistics (json) about the archive after it's compressed."
247247
)(
248-
"retain-float-format",
249-
po::bool_switch(&m_retain_float_format),
250-
"Store extra information to losslessly decompress floats."
248+
"no-retain-float-format",
249+
po::bool_switch(&m_no_retain_float_format),
250+
"Do not store extra information to losslessly decompress floats."
251251
)(
252252
"single-file-archive",
253253
po::bool_switch(&m_single_file_archive),

components/core/src/clp_s/CommandLineArguments.hpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,9 @@ class CommandLineArguments {
102102

103103
OutputHandlerType get_output_handler_type() const { return m_output_handler_type; }
104104

105-
[[nodiscard]] bool get_retain_float_format() const { return m_retain_float_format; }
105+
[[nodiscard]] auto get_retain_float_format() const -> bool {
106+
return false == m_no_retain_float_format;
107+
}
106108

107109
bool get_single_file_archive() const { return m_single_file_archive; }
108110

@@ -181,7 +183,7 @@ class CommandLineArguments {
181183
size_t m_target_encoded_size{8ULL * 1024 * 1024 * 1024}; // 8 GiB
182184
bool m_print_archive_stats{false};
183185
size_t m_max_document_size{512ULL * 1024 * 1024}; // 512 MB
184-
bool m_retain_float_format{false};
186+
bool m_no_retain_float_format{false};
185187
bool m_single_file_archive{false};
186188
bool m_structurize_arrays{false};
187189
bool m_ordered_decompression{false};

components/job-orchestration/job_orchestration/executor/compress/compression_task.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -290,7 +290,6 @@ def _make_clp_s_command_and_env(
290290
"--target-encoded-size",
291291
str(clp_config.output.target_segment_size + clp_config.output.target_dictionaries_size),
292292
"--compression-level", str(clp_config.output.compression_level),
293-
"--retain-float-format",
294293
]
295294
# fmt: on
296295

0 commit comments

Comments
 (0)