From 7dca1da8f2a03f580078698085b7e3195932bd97 Mon Sep 17 00:00:00 2001 From: Aleksei Antipovskii Date: Sun, 13 Jul 2025 17:25:29 +0200 Subject: [PATCH] fix(cpimport): MCOL-4882 fix -L arg default value --- writeengine/bulk/we_cmdargs.cpp | 7 +++++-- writeengine/splitter/we_cmdargs.cpp | 2 +- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/writeengine/bulk/we_cmdargs.cpp b/writeengine/bulk/we_cmdargs.cpp index 1989f0719..132771002 100644 --- a/writeengine/bulk/we_cmdargs.cpp +++ b/writeengine/bulk/we_cmdargs.cpp @@ -131,7 +131,7 @@ WECmdArgs::WECmdArgs(int argc, char** argv) "S3 Hostname (for S3 imports, Amazon's S3 default)") ("s3-region,g", po::value(&fS3Region), "S3 Region (for S3 imports)") - ("errors-dir,L", po::value(&fErrorDir)->default_value(MCSLOGDIR), + ("errors-dir,L", po::value(&fErrorDir)->default_value(fErrorDir), "Directory for the output .err and .bad files") ("job-uuid,u", po::value(&fUUID), "import job UUID") ("username,U", po::value(&fUsername), "Username of the files owner.") @@ -514,7 +514,10 @@ void WECmdArgs::fillParams(BulkLoad& curJob, std::string& sJobIdStr, std::string { cout << "number of read threads : " << fNoOfReadThrds << endl; } - cout << "Column delimiter : " << (fColDelim == '\t' ? "\\t" : string{fColDelim}) << endl; + if (fColDelim != 0) + { + cout << "Column delimiter : " << (fColDelim == '\t' ? "\\t" : string{fColDelim}) << endl; + } if (fNoOfWriteThrds != 0) { cout << "number of parse threads : " << fNoOfWriteThrds << endl; diff --git a/writeengine/splitter/we_cmdargs.cpp b/writeengine/splitter/we_cmdargs.cpp index 34f67ccb7..816e2a9f6 100644 --- a/writeengine/splitter/we_cmdargs.cpp +++ b/writeengine/splitter/we_cmdargs.cpp @@ -132,7 +132,7 @@ WECmdArgs::WECmdArgs(int argc, char** argv) "S3 Hostname (for S3 imports, Amazon's S3 default)") ("s3-region,g", po::value(&fS3Region), "S3 Region (for S3 imports)") - ("errors-dir,L", po::value(&fErrorDir)->default_value(MCSLOGDIR), + ("errors-dir,L", po::value(&fErrorDir)->default_value(fErrorDir), "Directory for the output .err and .bad files") ("username,U", po::value(&fUsername), "Username of the files owner.") ("dbname", po::value(), "Name of the database to load")