Skip to content

Commit 29defa3

Browse files
[FSSDK-11177] review update
1 parent 01b8f90 commit 29defa3

File tree

4 files changed

+8
-8
lines changed

4 files changed

+8
-8
lines changed

OptimizelySDK.Tests/CmabTests/DecisionServiceCmabTest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ public void TestGetVariationWithCmabExperimentServiceError()
179179

180180
Assert.IsNotNull(result);
181181
Assert.IsNull(result.ResultObject.Variation, "Should return null on error");
182-
Assert.IsTrue(result.ResultObject.CmabError);
182+
Assert.IsTrue(result.ResultObject.Error);
183183

184184
var reasonsList = result.DecisionReasons.ToReport(true);
185185

OptimizelySDK/Bucketing/DecisionService.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -812,7 +812,7 @@ public virtual Result<FeatureDecision> GetVariationForFeatureExperiment(
812812
#if USE_CMAB
813813
cmabUuid = variationResult?.CmabUuid;
814814

815-
if (variationResult?.CmabError == true)
815+
if (variationResult?.Error == true)
816816
{
817817
Logger.Log(LogLevel.ERROR,
818818
reasons.AddInfo(

OptimizelySDK/Bucketing/VariationDecisionResult.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,12 +24,12 @@ namespace OptimizelySDK.Bucketing
2424
public class VariationDecisionResult
2525
{
2626
public VariationDecisionResult(Variation variation, string cmabUuid = null,
27-
bool cmabError = false
27+
bool error = false
2828
)
2929
{
3030
Variation = variation;
3131
CmabUuid = cmabUuid;
32-
CmabError = cmabError;
32+
Error = error;
3333
}
3434

3535
/// <summary>
@@ -43,9 +43,9 @@ public VariationDecisionResult(Variation variation, string cmabUuid = null,
4343
public string CmabUuid { get; set; }
4444

4545
/// <summary>
46-
/// Indicates whether an error occurred during the CMAB decision process.
47-
/// False for non-CMAB experiments or successful CMAB decisions.
46+
/// Indicates whether an error occurred during the decision process.
47+
/// False for successful decisions or when no error occurred.
4848
/// </summary>
49-
public bool CmabError { get; set; }
49+
public bool Error { get; set; }
5050
}
5151
}

OptimizelySDK/Cmab/CmabConstants.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ internal static class CmabConstants
3535
"Failed to fetch CMAB data for experiment {0}.";
3636

3737
public static readonly TimeSpan MAX_TIMEOUT = TimeSpan.FromSeconds(10);
38-
public static readonly TimeSpan DEFAULT_CACHE_TTL = TimeSpan.FromMinutes(10);
38+
public static readonly TimeSpan DEFAULT_CACHE_TTL = TimeSpan.FromMinutes(30);
3939

4040
public const int CMAB_MAX_RETRIES = 1;
4141
public static readonly TimeSpan CMAB_INITIAL_BACKOFF = TimeSpan.FromMilliseconds(100);

0 commit comments

Comments
 (0)