Benchmark cache #955
- Loading branch information
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,38 @@ | ||
| <?php /** @noinspection SqlNoDataSourceInspection */ | ||
| use DBA\Factory; | ||
|
|
||
| if (!Util::databaseTableExists("HardwareGroup")) { | ||
|
There was a problem hiding this comment. Typically, we wrap upgrade steps also with in a check of a unique key which is stored in database to determine if the update part was already executed or not. Of course, if databaseTableExists() is checked, it would not execute it twice anyway. But with having a key stored in the database, it's also easier to look up in case of issues and to keep consistency overall, we should always have this wrap with a key (as seen in the other update files). |
||
| Factory::getAgentFactory()->getDB()->query("CREATE TABLE `HardwareGroup` ( | ||
| `hardwareGroupId` INT(11) NOT NULL, | ||
| `devices` VARCHAR(65000) NOT NULL, | ||
| ) ENGINE=InnoDB;" | ||
| ); | ||
| Factory::getAgentFactory()->getDB()->query("ALTER TABLE `HardwareGroup` MODIFY `hardwareGroupId` int(11) NOT NULL AUTO_INCREMENT"); | ||
| } | ||
|
|
||
| //change Agent table | ||
|
There was a problem hiding this comment. I guess this should not be commented out? Otherwise I think this would break existing installations as the Agent table would not be as expected with the benchmark caching code. The same probably applies for the Benchmark table and the adding of the foreign key constraints. |
||
| Factory::getAgentFactory()->getDB()->query("ALTER TABLE `Agent` | ||
| DROP COLUMN devices; | ||
| "); | ||
| Factory::getAgentFactory()->getDB()->query("ALTER TABLE `Agent` | ||
| ADD COLUMN hardwareGroupId INT(11); | ||
| "); | ||
| Factory::getAgentFactory()->getDB()->query("ALTER TABLE `Agent` ADD CONSTRAINT FOREIGN KEY (`hardwareGroupId`) REFERENCES `HardwareGroup` (`hardwareGroupId`);"); | ||
|
|
||
|
|
||
| if (!Util::databaseTableExists("Benchmark")) { | ||
| Factory::getAgentFactory()->getDB()->query("CREATE TABLE `Benchmark` ( | ||
| `benchmarkId` INT(11) NOT NULL, | ||
| `benchmarkValue` VARCHAR(256) NOT NULL, | ||
| `hardwareGroupId` INT(11) NOT NULL, | ||
| `attackParameters` VARCHAR(256) NOT NULL, | ||
| `ttl` INT(11) NULL, | ||
| `hashMode` INT(11) NULL, | ||
| `benchmarkType` VARCHAR(10) NULL, | ||
| ) ENGINE=InnoDB;" | ||
| ); | ||
| Factory::getAgentFactory()->getDB()->query("ALTER TABLE `Benchmark` MODIFY `benchmarkId` int(11) NOT NULL AUTO_INCREMENT"); | ||
| } | ||
| Factory::getAgentFactory()->getDB()->query("ALTER TABLE `Benchmark` ADD CONSTRAINT FOREIGN KEY (`hardwareGroupId`) REFERENCES `HardwareGroup` (`hardwareGroupId`);"); | ||
|
|
||
| ?> | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While the release is not fixed and in preparation, the update files typically are named ./update_v0.14.x_v0.x.x.php.