Elgato_dark/fleet
1
0
Fork 0
mirror of https://github.com/fleetdm/fleet synced 2026-05-21 16:08:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 100

Drop attempt number from index to avoid extra write overhead (#38391)

Browse source 
Found in connection with #35916. No changes file as this is effectively
an unreleased bugfix (will cherry-pick this). No updates to tests
because tests don't test index existence.

- [x] Input data is properly validated, `SELECT *` is avoided, SQL
injection is prevented (using placeholders for values in statements)
This commit is contained in:
Ian Littman 2026-01-15 12:56:01 -06:00 committed by GitHub
parent 705663f826
commit d428e2c7fc
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
server/datastore/mysql/migrations/tables/20260109231821_AddAttemptNumberToScriptsAndSoftwareInstalls.go
View file

@ -28,14 +28,14 @@ func Up_20260109231821(tx *sql.Tx) error {
_, err = tx.Exec(`
ALTER TABLE host_script_results
ADD INDEX idx_host_script_results_host_policy_attempt (host_id, policy_id, attempt_number);
ADD INDEX idx_host_script_results_host_policy (host_id, policy_id);
`)
if err != nil {
return errors.Wrap(err, "adding index to host_script_results")
}
_, err = tx.Exec(`
ALTER TABLE host_software_installs
ADD INDEX idx_host_software_installs_host_policy_attempt (host_id, policy_id, attempt_number);
ADD INDEX idx_host_software_installs_host_policy (host_id, policy_id);
`)
if err != nil {
return errors.Wrap(err, "adding index to host_software_installs")