In reporter.c there is a block of code at line 1740 which looks like:

    if(curprog != P_PLANNER) {
        if(curprog != P_DUMPER ||
           sscanf(s - 1,"[sec %f kb %f kps %f orig-kb %f", 
                  &sec, &kbytes, &kps, &origkb) != 4)  {
            origkb = -1;
            if(sscanf(s - 1,"[sec %f kb %f kps %f",
                      &sec, &kbytes, &kps) != 3) {
                bogus_line();
                return;
            }
        }
        else {
            if(origkb == 0.0) origkb = 0.1;
        }
    }

Should that test in the second "if" really be "!= P_DUMPER"?  I would
think "== P_DUMPER" would be much more useful.

Also, my backups' compressed size and compression rates don't seem to be
getting stored in the info files:

version: 0
command: 0
full-rate: 173.000000
full-comp: 0.349281
incr-rate: 1.000000 1.000000 1.000000
incr-comp: 0.000000 0.200000 0.200000
stats: 0 1392115 486240 2798 972179254 6 Daily04
stats: 1 67423 0 178 972472938 4 Daily08
stats: 2 71700 0 165 972817708 5 Daily16
last_level: 2 6

Yet for that disk, the log shows:

SUCCESS dumper localhost / 20001029 2 [sec 164.844 kb 14784 kps 89.7 orig-kb 71700]
SUCCESS chunker localhost / 20001029 2 [sec 165.063 kb 14784 kps 89.8]
SUCCESS taper localhost / 20001029 2 [sec 20.035 kb 14784 kps 737.9 {wr: writers 463 
rdwait 0.053 wrwait 19.902 filemark 0.017}]

So there certainly was a compressed value and a compression rate should
be calculated.

I will hunt the info file bug down once I have the reporter working.

b.


-- 
Brian J. Murrell

Reply via email to