Skip to content

Commit 2e2dfc5

Browse files
committed
[hadd] change default verbosity from 99 to 2
1 parent 864029e commit 2e2dfc5

File tree

9 files changed

+6
-13
lines changed

9 files changed

+6
-13
lines changed

main/src/hadd.cxx

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,8 @@
174174
// gHaddVerbosity = 1: only print hadd errors + warnings
175175
// gHaddVerbosity = 2: print hadd errors + warnings and TFileMerger messages
176176
// gHaddVerbosity > 2: print all hadd and TFileMerger messages.
177-
static int gHaddVerbosity = 2;
177+
static constexpr int kDefaultHaddVerbosity = 2;
178+
static int gHaddVerbosity = kDefaultHaddVerbosity;
178179

179180
namespace {
180181

@@ -571,7 +572,7 @@ static std::optional<HAddArgs> ParseArgs(int argc, char **argv)
571572
PARSE_FLAG(FlagArg, argc, argv, argIdx, "cachesize", args.fCacheSize, {}, ConvertCacheSize);
572573
PARSE_FLAG(FlagArg, argc, argv, argIdx, "experimental-io-features", args.fFeatures);
573574
PARSE_FLAG(FlagArg, argc, argv, argIdx, "n", args.fMaxOpenedFiles);
574-
PARSE_FLAG(FlagArg, argc, argv, argIdx, "v", args.fVerbosity, {99});
575+
PARSE_FLAG(FlagArg, argc, argv, argIdx, "v", args.fVerbosity, {kDefaultHaddVerbosity});
575576
PARSE_FLAG(FlagF, arg, args);
576577

577578
#undef PARSE_FLAG
@@ -664,7 +665,7 @@ int main(int argc, char **argv)
664665

665666
ROOT::TIOFeatures features = args.fFeatures.value_or(ROOT::TIOFeatures{});
666667
Int_t maxopenedfiles = args.fMaxOpenedFiles.value_or(0);
667-
gHaddVerbosity = args.fVerbosity.value_or(99);
668+
gHaddVerbosity = args.fVerbosity.value_or(kDefaultHaddVerbosity);
668669
Int_t newcomp = args.fCompressionSettings.value_or(-1);
669670
TString cacheSize = args.fCacheSize.value_or("");
670671

@@ -682,7 +683,7 @@ int main(int argc, char **argv)
682683
nProcesses = s.fCpus;
683684
}
684685
if (multiproc)
685-
Info(3) << "parallelizing with " << nProcesses << " processes.\n";
686+
Info(2) << "parallelizing with " << nProcesses << " processes.\n";
686687

687688
// If the user specified a workingDir, use that. Otherwise, default to the system temp dir.
688689
std::string workingDir;
@@ -715,7 +716,7 @@ int main(int argc, char **argv)
715716
Info(2) << "target file: " << targetname << "\n";
716717

717718
if (args.fCacheSize)
718-
Info(3) << "Using " << cacheSize << "\n";
719+
Info(2) << "Using " << cacheSize << "\n";
719720

720721
////////////////////////////// end flags processing /////////////////////////////////
721722

roottest/root/io/evolution/versions/attmarker.ref

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,3 @@ hadd Source file 2: hsimple20.root
55
Warning in <hadd>: Sources and Target have different compression settings
66
hadd merging will be slower
77
hadd Target path: result.root:/
8-
Info in <hadd>: merged 2 (2) input (partial) files into result.root

roottest/root/io/hadd/hadd_args_cachesize.ref

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,3 @@ Info in <hadd>: Using cachesize=100M
33
Info in <hadd>: compression setting for all output: 101
44
hadd Source file 1: hadd_args_in.root
55
hadd Target path: hadd_args_cachesize_out.root:/
6-
Info in <hadd>: merged 1 (1) input (partial) files into hadd_args_cachesize_out.root

roottest/root/io/hadd/hadd_args_ff_dbg_j_2.ref

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,3 @@ Info in <hadd>: each process should handle at least 3 files for efficiency. Sett
55
hadd Source file 1: hadd_args_in.root
66
hadd Source file 2: hadd_args_in.root
77
hadd Target path: hadd_args_ff_dbg_j_2_out.root:/
8-
Info in <hadd>: merged 2 (2) input (partial) files into hadd_args_ff_dbg_j_2_out.root

roottest/root/io/hadd/hadd_args_minusminus.ref

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,3 @@ hadd Source file 1: hadd_args_in.root
44
Warning in <hadd>: Sources and Target have different compression settings
55
hadd merging will be slower
66
hadd Target path: --hadd_args_out.root:/
7-
Info in <hadd>: merged 1 (1) input (partial) files into --hadd_args_out.root

roottest/root/io/hadd/hadd_args_n0.ref

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,3 @@ Info in <hadd>: target file: hadd_args_n0_out.root
22
Info in <hadd>: compression setting for all output: 101
33
hadd Source file 1: hadd_args_in.root
44
hadd Target path: hadd_args_n0_out.root:/
5-
Info in <hadd>: merged 1 (1) input (partial) files into hadd_args_n0_out.root

roottest/root/io/hadd/hadd_args_n_1.ref

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,3 @@ Info in <hadd>: target file: hadd_args_n_1_out.root
22
Info in <hadd>: compression setting for all output: 101
33
hadd Source file 1: hadd_args_in.root
44
hadd Target path: hadd_args_n_1_out.root:/
5-
Info in <hadd>: merged 1 (1) input (partial) files into hadd_args_n_1_out.root

roottest/root/io/newstl/emulatedWrite.ref

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,3 @@ Warning in <TClass::Init>: no dictionary for class CaloJet is available
1717
Warning in <TClass::Init>: no dictionary for class edm::EDCollection<CaloJet> is available
1818
hadd Source file 2: cmsJets_1.root
1919
hadd Target path: result.root:/
20-
Info in <hadd>: merged 2 (2) input (partial) files into result.root

roottest/root/io/transient/base/hadd_autoloadZLIB.ref

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,3 @@ Info in <hadd>: compression setting for all output: 101
33
hadd Source file 1: data1.root
44
hadd Source file 2: data2.root
55
hadd Target path: data_merge.root:/
6-
Info in <hadd>: merged 2 (2) input (partial) files into data_merge.root

0 commit comments

Comments
 (0)