Skip to content

Commit b62194d

Browse files
committed
Allow ExpirationProcessTimerInSeconds to be set to 36 hours to improve compatibility with RavenDB cloud.
1 parent b0d09fa commit b62194d

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

src/ServiceControl.Audit.Persistence.RavenDB/RavenPersistenceConfiguration.cs

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ static int GetExpirationProcessTimerInSeconds(PersistenceSettings settings)
142142
expirationProcessTimerInSeconds = int.Parse(expirationProcessTimerInSecondsString);
143143
}
144144

145-
var maxExpirationProcessTimerInSeconds = TimeSpan.FromHours(3).TotalSeconds;
145+
var maxExpirationProcessTimerInSeconds = TimeSpan.FromHours(36).TotalSeconds; // Compatibility with RavenDB cloud free license
146146

147147
if (expirationProcessTimerInSeconds < 0)
148148
{
@@ -156,6 +156,13 @@ static int GetExpirationProcessTimerInSeconds(PersistenceSettings settings)
156156
return ExpirationProcessTimerInSecondsDefault;
157157
}
158158

159+
var warningThresholdExpirationProcessTimerInSeconds = TimeSpan.FromHours(3).TotalSeconds;
160+
161+
if (expirationProcessTimerInSeconds > warningThresholdExpirationProcessTimerInSeconds)
162+
{
163+
Logger.LogError("It is not recommended to set `ExpirationProcessTimerInSeconds` above {WarningThresholdExpirationProcessTimerInSeconds} set ", warningThresholdExpirationProcessTimerInSeconds);
164+
}
165+
159166
return expirationProcessTimerInSeconds;
160167
}
161168

0 commit comments

Comments
 (0)