Use optimized inner join to mark parent as outdated

Signed-off-by: Ari Selseng <ari@selseng.net>

Author: ariselseng

apps/files_external/lib/Command/Notify.php

@@ -161,14 +161,15 @@ private function markParentAsOutdated($mountId, $path, OutputInterface $output)
 			$parent = '';
 		}
 
+		$pathHash = md5(trim(\OC_Util::normalizeUnicode($parent), '/'));
 		try {
-			$this->updateQuery->execute([$parent, $mountId]);
+			$this->updateQuery->execute([$mountId, $pathHash]);
 		} catch (DriverException $ex) {
 			$this->logger->logException($ex, ['app' => 'files_external', 'message' => 'Error while trying to mark folder as outdated', 'level' => ILogger::WARN]);
 			$this->connection = $this->reconnectToDatabase($this->connection, $output);
 			$output->writeln('<info>Needed to reconnect to the database</info>');
 			$this->updateQuery = $this->getUpdateQuery($this->connection);
-			$this->updateQuery->execute([$parent, $mountId]);
+			$this->updateQuery->execute([$mountId, $pathHash]);
 		}
 	}
 
@@ -204,9 +205,11 @@ private function logUpdate(IChange $change, OutputInterface $output) {
 	private function getUpdateQuery(IDBConnection $connection) {
 		// the query builder doesn't really like subqueries with parameters
 		return $connection->prepare(
-			'UPDATE *PREFIX*filecache SET size = -1
-			WHERE `path` = ?
-			AND `storage` IN (SELECT storage_id FROM *PREFIX*mounts WHERE mount_id = ?)'
+			'UPDATE *PREFIX*filecache
+			INNER JOIN *PREFIX*mounts
+			ON (*PREFIX*filecache.storage = *PREFIX*mounts.storage_id)
+			SET size = -1
+			WHERE `path_hash` = ?'
 		);
 	}