From 8296fe6cba2dc83571fb17274690952b42aa6758 Mon Sep 17 00:00:00 2001 From: Ruben Cerna Date: Fri, 5 Jun 2026 17:04:22 -0700 Subject: [PATCH] Remove excess logs --- .../MsSqlMetadataProvider.cs | 25 +++++++++++++------ .../MetadataProviders/SqlMetadataProvider.cs | 19 ++++++++------ 2 files changed, 28 insertions(+), 16 deletions(-) diff --git a/src/Core/Services/MetadataProviders/MsSqlMetadataProvider.cs b/src/Core/Services/MetadataProviders/MsSqlMetadataProvider.cs index cdb54a2ac2..93cbb06e51 100644 --- a/src/Core/Services/MetadataProviders/MsSqlMetadataProvider.cs +++ b/src/Core/Services/MetadataProviders/MsSqlMetadataProvider.cs @@ -81,13 +81,19 @@ public override async Task PopulateTriggerMetadataForTable(string entityName, st if ("UPDATE".Equals(type_desc)) { sourceDefinition.IsUpdateDMLTriggerEnabled = true; - _logger.LogInformation($"An update trigger is enabled for the entity: {entityName}"); + if (!_isValidateOnly) + { + _logger.LogInformation($"An update trigger is enabled for the entity: {entityName}"); + } } if ("INSERT".Equals(type_desc)) { sourceDefinition.IsInsertDMLTriggerEnabled = true; - _logger.LogInformation($"An insert trigger is enabled for the entity: {entityName}"); + if (!_isValidateOnly) + { + _logger.LogInformation($"An insert trigger is enabled for the entity: {entityName}"); + } } } } @@ -362,13 +368,16 @@ protected override async Task GenerateAutoentitiesIntoEntities(IReadOnlyDictiona subStatusCode: DataApiBuilderException.SubStatusCodes.ErrorInInitialization); } - if (runtimeConfig.IsRestEnabled) + if (!_isValidateOnly) { - _logger.LogInformation("[{entity}] REST path: {globalRestPath}/{entityRestPath}", entityName, runtimeConfig.RestPath, entityName); - } - else - { - _logger.LogInformation(message: "REST calls are disabled for the entity: {entity}", entityName); + if (runtimeConfig.IsRestEnabled) + { + _logger.LogInformation("[{entity}] REST path: {globalRestPath}/{entityRestPath}", entityName, runtimeConfig.RestPath, entityName); + } + else + { + _logger.LogInformation(message: "REST calls are disabled for the entity: {entity}", entityName); + } } addedEntities++; diff --git a/src/Core/Services/MetadataProviders/SqlMetadataProvider.cs b/src/Core/Services/MetadataProviders/SqlMetadataProvider.cs index d93b4edbcf..888663ebe1 100644 --- a/src/Core/Services/MetadataProviders/SqlMetadataProvider.cs +++ b/src/Core/Services/MetadataProviders/SqlMetadataProvider.cs @@ -123,16 +123,19 @@ public SqlMetadataProvider( _databaseType = runtimeConfig.GetDataSourceFromDataSourceName(dataSourceName).DatabaseType; _logger = logger; _isValidateOnly = isValidateOnly; - foreach ((string entityName, Entity entityMetatdata) in Entities) + if (!_isValidateOnly) { - if (runtimeConfig.IsRestEnabled) + foreach ((string entityName, Entity entityMetatdata) in Entities) { - string restPath = entityMetatdata.Rest?.Path ?? entityName; - _logger.LogInformation("[{entity}] REST path: {globalRestPath}/{entityRestPath}", entityName, runtimeConfig.RestPath, restPath); - } - else - { - _logger.LogInformation(message: "REST calls are disabled for the entity: {entity}", entityName); + if (runtimeConfig.IsRestEnabled) + { + string restPath = entityMetatdata.Rest?.Path ?? entityName; + _logger.LogInformation("[{entity}] REST path: {globalRestPath}/{entityRestPath}", entityName, runtimeConfig.RestPath, restPath); + } + else + { + _logger.LogInformation(message: "REST calls are disabled for the entity: {entity}", entityName); + } } }