3.0.6-3.1.0-upgrade-db.sql - item_lang attribute reingest redundant

[Josh Stompro]

Hello, the 1094 update in the 3.0.6-3.1.0-upgrade-db.sql looks like it is redundant since the script ends in a full reingest.

On my test system it took ~40 minutes for that update to run, it touches about 15K records out of 240K. The last pingest.pl run took 3.4 hours on that system. So it might make sense to save time and skip that update.

I doubled checked after a full reingest it appears as if there are more item_lang attributes mapped for records with 041 abdefgm values.

http://git.evergreen-ils.org/?p=Evergreen.git;a=blob;f=Open-ILS/src/sql/Pg/version-upgrade/3.0.6-3.1.0-upgrade-db.sql;h=bc5e5c9868101ff55f7965c16ebe88cdd87b5185;hb=refs/heads/tags/rel_3_1_1#l552

I'm not sure about the correct way to skip an update like that. Does the "SELECT evergreen.upgrade_deps_block_check('1094', :eg_version);" still get run, but just skip the reingest command?

Josh