Support Center

Name is required.
Email address is required.
Invalid email address
Answer is required.
Exceeding max length of 5KB

Minor issue in MinionReindexInstall.sql

Marc Brooks May 24, 2016 05:04PM CDT

I was reviewing the most-excellent MR and noticed that in 1.2 upgrade installation path, down along line #508 we add the CONSTRAINT ckRebuildGTReorg with

ALTER TABLE Minion.IndexSettingsDB ADD CONSTRAINT
ckRebuildGTReorg CHECK (([RebuildThreshold]>[ReorgThreshold]))

but by line #511 (where we COMMIT), we haven't actually checked the new CONSTRAINT, so it wouldn't be a trusted constraint. Shouldn't we before the COMMIT do this?

ALTER TABLE Minion.IndexSettingsDB CHECK CONSTRAINT ckRebuildGTReorg

Just wondering...

Up 0 rated Down
Sean McCown Jul 18, 2016 03:27PM CDT MinionWare Agent

Thanks for letting us know. We’re taking a look at this for the next service release.

Post Your Public Answer

Your name (required)
Your email address (required)
Answer (required)
556ca399015f31edc97a62de2771be1a@minionware.desk-mail.com
https://cdn.desk.com/
false
desk
Loading
seconds ago
a minute ago
minutes ago
an hour ago
hours ago
a day ago
days ago
about
false
Invalid characters found
/customer/en/portal/articles/autocomplete