From 346fcd068cc08ba8f39c6af79b6476025c8daa93 Mon Sep 17 00:00:00 2001 From: Christoph Weitkamp Date: Tue, 4 Jan 2022 21:26:52 +0100 Subject: [PATCH] Do not create table on 'query', only on 'store' (#11961) Signed-off-by: Christoph Weitkamp --- .../jdbc/internal/JdbcPersistenceService.java | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/bundles/org.openhab.persistence.jdbc/src/main/java/org/openhab/persistence/jdbc/internal/JdbcPersistenceService.java b/bundles/org.openhab.persistence.jdbc/src/main/java/org/openhab/persistence/jdbc/internal/JdbcPersistenceService.java index 44e584303a..bf8103d22b 100644 --- a/bundles/org.openhab.persistence.jdbc/src/main/java/org/openhab/persistence/jdbc/internal/JdbcPersistenceService.java +++ b/bundles/org.openhab.persistence.jdbc/src/main/java/org/openhab/persistence/jdbc/internal/JdbcPersistenceService.java @@ -193,19 +193,16 @@ public class JdbcPersistenceService extends JdbcMapper implements QueryablePersi String table = sqlTables.get(itemName); if (table == null) { - logger.warn( - "JDBC::query: unable to find table for query, no data in database for item '{}'. Current number of tables in the database: {}", - itemName, sqlTables.size()); - // if enabled, table will be created immediately - logger.warn("JDBC::query: try to generate the table for item '{}'", itemName); - table = getTable(item); + logger.debug("JDBC::query: unable to find table for item with name: '{}', no data in database.", itemName); + return List.of(); } long timerStart = System.currentTimeMillis(); List items = getHistItemFilterQuery(filter, conf.getNumberDecimalcount(), table, item); - - logger.debug("JDBC::query: query for {} returned {} rows in {} ms", itemName, items.size(), - System.currentTimeMillis() - timerStart); + if (logger.isDebugEnabled()) { + logger.debug("JDBC: Query for item '{}' returned {} rows in {} ms", itemName, items.size(), + System.currentTimeMillis() - timerStart); + } // Success errCnt = 0; -- 2.47.3