Please send source patch re: binary parallel load errors


#1

Glad I didn’t start bulk loading yet.
Just read the thread http://www.scidb.org/forum/viewtopic.php?f=11&t=1536

Mike, if you have source patches, would you send them my way too?

Opaque save and reload from 14.8 to 14.12 won’t help me, I have to use the binary load files that I already created.
Thanks, George


#2

Hi. I’ve gotten feedback that tossing source patches around on the forum is frowned upon, so instead I have made sure that the 14.12 source tarball is up-to-date to reflect the fix. A link to the new source tarball scidb-14.12.0.8739.tgz is now at the bottom of the release information page: scidb.org/forum/viewtopic.php?f=14&t=1526 . The older (and broken) scidb-14.12.0.8698.tgz tarball is no longer available. Within the new tarball, the files that changed to fix this issue were:

[ul]src/query/ops/input/ChunkLoader.cpp
src/query/ops/input/CsvChunkLoader.cpp
src/query/ops/save/PhysicalSave.cpp
src/smgr/io/ArrayWriter.h
src/smgr/io/ArrayWriter.cpp
utils/iquery/iquery.cpp[/ul]
A new regression test was also added:

[ul]tests/harness/testcases/t/pload/paraload2.test
tests/harness/testcases/t/pload/paraload2.expected[/ul]
If you had a source tree created from the previous 14.12 tarball, simply replacing those files should do the trick.