localpcguy / cbloader

Automatically exported from code.google.com/p/cbloader
1 stars 0 forks source link

cbloader merges parts on load every time #32

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Install CBLoader 1.1.6 with default config
2. Run more than once

What is the expected output? What do you see instead?
Expected less part merging; now merges every time, regardless of file content.

What version of the product are you using? On what operating system?
1.1.6 on Windows 7 Pro x64 edition.

Please provide any additional information below.
Using supplied default config file; is this related to the "fastmode" setting 
being set to "false"?

Original issue reported on code.google.com by mr.azrae...@gmail.com on 2 Jan 2011 at 5:37

GoogleCodeExporter commented 9 years ago
I'm still trying to work out how Jeff's implementation of this feature works. 
(Jeff: Comment the code please).  

It appears to work with a low volume of files, but fails with larger amounts.

Original comment by YeOldeSt...@gmail.com on 2 Jan 2011 at 11:04

GoogleCodeExporter commented 9 years ago
This appears to be working just fine for me. It's likely you're hitting a bug 
that I just don't have the data files to reproduce.

The new workflow is as follows:

The first time with a new file: Merge Everything
The first modification to 1 file: Merge Everything
Subsequent modifications to that same file: Only merge the changed file
Subsequent modifications to any other file: Merge everything

This is ideal if you're repeatedly modifying the same file.4

That said, it sounds like Steve has reproduced some issue. Perhaps it has to do 
with some kind of filename in one of the part files you're working with? Can 
you play with it a bit and see if you can narrow it down to a specific part 
file that is forcing a re-merge? 

Original comment by jeff.h...@gmail.com on 3 Jan 2011 at 7:13

GoogleCodeExporter commented 9 years ago
I'm considering pulling this code out for the moment.  It seems more trouble 
than it's worth.

Original comment by YeOldeSt...@gmail.com on 19 Jun 2011 at 6:20

GoogleCodeExporter commented 9 years ago
Disabled the code in 1.3.0.  Using the new method, it's actually slower to 
split it into two groups.

Original comment by YeOldeSt...@gmail.com on 20 Aug 2011 at 6:15