removed query store on checks (#252)

* removed query store on checks

* added changelog

Author: tanushangural

CHANGELOG.md

@@ -9,6 +9,9 @@ Unreleased section should follow [Release Toolkit](https://github.com/newrelic/r
 
 ## Unreleased
 
+### enhancements
+- Removed the query store on/off check in prevalidations and queries in qpm
+
 ## v2.21.1 - 2025-09-18
 
 ### 🐞 Bug fixes

src/queryanalysis/config/query_config.go

@@ -87,7 +87,7 @@ var Queries = []models.QueryDetailsDto{
 						AND EXISTS (
 							SELECT 1
 							FROM sys.databases d
-							WHERE d.database_id = CONVERT(INT, pa.value) AND d.is_query_store_on = 1
+							WHERE d.database_id = CONVERT(INT, pa.value)
 						)
 				)
 				SELECT
@@ -149,7 +149,6 @@ var Queries = []models.QueryDetailsDto{
 				DECLARE db_cursor CURSOR FOR
 				SELECT name FROM sys.databases
 				WHERE state_desc = 'ONLINE'
-				AND is_query_store_on = 1
 				AND database_id > 4;
 				
 				OPEN db_cursor;
@@ -280,7 +279,6 @@ var Queries = []models.QueryDetailsDto{
 				OUTER APPLY sys.dm_exec_sql_text(blocking_info.blocked_sql_handle) AS blocked_sql
 				OUTER APPLY sys.dm_exec_input_buffer(blocking_info.blocking_spid, NULL) AS input_buffer
 				JOIN sys.databases AS db ON db.database_id = blocking_info.database_id
-				WHERE db.is_query_store_on = 1
 				ORDER BY
     				blocking_info.start_time;`,
 		Type: "blockingSessions",

src/queryanalysis/validation/validation_check.go

@@ -25,7 +25,7 @@ func ValidatePreConditions(sqlConnection *connection.SQLConnection) bool {
 		return false
 	}
 
-	if !checkDatabaseCompatibility(databaseDetails) || !queryStoresAreEnabledForAnyDB(databaseDetails) {
+	if !checkDatabaseCompatibility(databaseDetails) {
 		return false
 	}
 
@@ -75,19 +75,3 @@ func checkPermissionsAndLogin(sqlConnection *connection.SQLConnection) bool {
 	}
 	return true
 }
-
-func queryStoresAreEnabledForAnyDB(databaseDetails []models.DatabaseDetailsDto) bool {
-	allQueryStoresOff := true
-	for _, database := range databaseDetails {
-		if database.IsQueryStoreOn {
-			log.Debug("Query store is enabled for this database %s. Please ensure that the query capture mode is set to \"ALL\" to capture all queries.", database.Name)
-			allQueryStoresOff = false
-		} else {
-			log.Debug("Query store is disabled for this database %s, so query monitoring will be skipped. To enable it, use the following command: `ALTER DATABASE %s SET QUERY_STORE = ON (QUERY_CAPTURE_MODE = ALL);", database.Name, database.Name)
-		}
-	}
-	if allQueryStoresOff {
-		log.Error("Query store is currently turned off for all databases, so query monitoring will be skipped. To enable it and set the capture mode to \"ALL,\" please refer to this guide: [New Relic Documentation for Microsoft SQL](https://docs.newrelic.com/install/microsoft-sql/).")
-	}
-	return !allQueryStoresOff
-}