Handle DBNull to fix log issues (#2191)

microsoft · Dec 16, 2022 · 90bc0fd · 90bc0fd
1 parent 1ebe9a7
commit 90bc0fd
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/Microsoft.Health.Dicom.SqlServer/Features/ChangeFeed/SqlChangeFeedStoreV24.cs b/src/Microsoft.Health.Dicom.SqlServer/Features/ChangeFeed/SqlChangeFeedStoreV24.cs
@@ -76,8 +76,8 @@ public override async Task<long> GetMaxDeletedChangeFeedWatermarkAsync(DateTime
         using SqlCommandWrapper sqlCommandWrapper = sqlConnectionWrapper.CreateRetrySqlCommand();
 
         VLatest.GetMaxDeletedChangeFeedWatermark.PopulateCommand(sqlCommandWrapper, timeStamp);
-        var maxSequence = (long?)(await sqlCommandWrapper.ExecuteScalarAsync(cancellationToken));
+        var maxSequence = await sqlCommandWrapper.ExecuteScalarAsync(cancellationToken);
 
-        return maxSequence.GetValueOrDefault();
+        return maxSequence != null && maxSequence != DBNull.Value ? (long)maxSequence : default;
     }
 }