bot comments

Author: tanderson-ld

lib/sdk/server/src/main/java/com/launchdarkly/sdk/server/integrations/TestData.java

@@ -281,6 +281,8 @@ public static final class FlagBuilder {
       this.on = from.on;
       this.fallthroughVariation = from.fallthroughVariation;
       this.variations = new CopyOnWriteArrayList<>(from.variations);
+      this.samplingRatio = from.samplingRatio;
+      this.migrationCheckRatio = from.migrationCheckRatio;
       for (ContextKind contextKind: from.targets.keySet()) {
         this.targets.put(contextKind, new TreeMap<>(from.targets.get(contextKind)));
       }

lib/sdk/server/src/main/java/com/launchdarkly/sdk/server/integrations/TestDataV2.java

@@ -180,10 +180,6 @@ public TestDataV2 update(TestData.FlagBuilder flagBuilder) {
    * synchronizer is having problems (for example, a network failure interrupting the streaming connection). It
    * does not actually stop the {@code TestDataV2} synchronizer from working, so even if you have simulated
    * an outage, calling {@link #update(TestData.FlagBuilder)} will still send updates.
-   * <p>
-   * The mapping from legacy {@link DataSourceStatusProvider.State} to FDv2 status results matches
-   * {@link com.launchdarkly.sdk.server.DataSourceSynchronizerAdapter}: OFF with error → terminalError,
-   * OFF with null → shutdown, INTERRUPTED → interrupted, VALID/INITIALIZING → no event.
    *
    * @param newState one of the constants defined by {@link com.launchdarkly.sdk.server.interfaces.DataSourceStatusProvider.State}
    * @param newError an {@link com.launchdarkly.sdk.server.interfaces.DataSourceStatusProvider.ErrorInfo} instance,