[AppConfig] Use the latest stable version 2023-10-01 and remove azure-core-experimental dependency (Azure#37472)

Author: mssfang

eng/versioning/version_client.txt

@@ -109,7 +109,7 @@ com.azure:azure-cosmos-encryption;2.6.0;2.7.0-beta.1
 com.azure:azure-cosmos-test;1.0.0-beta.6;1.0.0-beta.7
 com.azure:azure-cosmos-tests;1.0.0-beta.1;1.0.0-beta.1
 com.azure.cosmos.kafka:azure-cosmos-kafka-connect;1.0.0-beta.1;1.0.0-beta.1
-com.azure:azure-data-appconfiguration;1.4.10;1.5.0-beta.3
+com.azure:azure-data-appconfiguration;1.4.10;1.5.0
 com.azure:azure-data-appconfiguration-perf;1.0.0-beta.1;1.0.0-beta.1
 com.azure:azure-data-schemaregistry;1.4.0;1.5.0-beta.1
 com.azure:azure-data-schemaregistry-apacheavro;1.1.11;1.2.0-beta.2

sdk/appconfiguration/azure-data-appconfiguration-perf/pom.xml

@@ -32,7 +32,7 @@
     <dependency>
       <groupId>com.azure</groupId>
       <artifactId>azure-data-appconfiguration</artifactId>
-      <version>1.5.0-beta.3</version> <!-- {x-version-update;com.azure:azure-data-appconfiguration;current} -->
+      <version>1.5.0</version> <!-- {x-version-update;com.azure:azure-data-appconfiguration;current} -->
     </dependency>
 
     <dependency>

sdk/appconfiguration/azure-data-appconfiguration/CHANGELOG.md

@@ -1,15 +1,24 @@
 # Release History
 
-## 1.5.0-beta.3 (Unreleased)
+## 1.5.0 (2023-11-08)
 
 ### Features Added
 
+- Added support for the `2023-10-01` service version.
+
 ### Breaking Changes
 
-### Bugs Fixed
+Note: Below breaking changes only affect the version `1.5.0-beta.2`.
+
+- Removed `azure-core-experimental` as a dependency and replaced usage of `PollResult` by `PollOperationDetails`.
 
 ### Other Changes
 
+#### Dependency Updates
+
+- Upgraded `azure-core` from `1.44.1` to version `1.45.0`.
+- Upgraded `azure-core-http-netty` from `1.13.9` to version `1.13.10`.
+
 ## 1.4.10 (2023-10-20)
 
 ### Other Changes

sdk/appconfiguration/azure-data-appconfiguration/README.md

@@ -55,7 +55,7 @@ add the direct dependency to your project as follows.
 <dependency>
   <groupId>com.azure</groupId>
   <artifactId>azure-data-appconfiguration</artifactId>
-  <version>1.4.7</version>
+  <version>1.5.0</version>
 </dependency>
 ```
 [//]: # ({x-version-update-end})
@@ -461,7 +461,7 @@ String snapshotName = "{snapshotName}";
 List<ConfigurationSettingsFilter> filters = new ArrayList<>();
 // Key Name also supports RegExp but only support prefix end with "*", such as "k*" and is case-sensitive.
 filters.add(new ConfigurationSettingsFilter("Test*"));
-SyncPoller<PollResult, ConfigurationSnapshot> poller =
+SyncPoller<PollOperationDetails, ConfigurationSnapshot> poller =
     configurationClient.beginCreateSnapshot(snapshotName, new ConfigurationSnapshot(filters), Context.NONE);
 poller.setPollInterval(Duration.ofSeconds(10));
 poller.waitForCompletion();

sdk/appconfiguration/azure-data-appconfiguration/assets.json

@@ -2,5 +2,5 @@
   "AssetsRepo": "Azure/azure-sdk-assets",
   "AssetsRepoPrefixPath": "java",
   "TagPrefix": "java/appconfiguration/azure-data-appconfiguration",
-  "Tag": "java/appconfiguration/azure-data-appconfiguration_b0d3480df4"
+  "Tag": "java/appconfiguration/azure-data-appconfiguration_0d1870bb82"
 }

sdk/appconfiguration/azure-data-appconfiguration/pom.xml

@@ -13,7 +13,7 @@
 
   <groupId>com.azure</groupId>
   <artifactId>azure-data-appconfiguration</artifactId>
-  <version>1.5.0-beta.3</version> <!-- {x-version-update;com.azure:azure-data-appconfiguration;current} -->
+  <version>1.5.0</version> <!-- {x-version-update;com.azure:azure-data-appconfiguration;current} -->
 
   <name>Microsoft Azure client library for App Configuration</name>
   <description>This package contains the Microsoft Azure App Configuration client library.</description>
@@ -63,11 +63,6 @@
       <artifactId>azure-core-http-netty</artifactId>
       <version>1.13.10</version> <!-- {x-version-update;com.azure:azure-core-http-netty;dependency} -->
     </dependency>
-    <dependency>
-      <groupId>com.azure</groupId>
-      <artifactId>azure-core-experimental</artifactId>
-      <version>1.0.0-beta.46</version> <!-- {x-version-update;com.azure:azure-core-experimental;dependency} -->
-    </dependency>
     <!-- Test dependencies -->
     <dependency>
       <groupId>com.azure</groupId>

sdk/appconfiguration/azure-data-appconfiguration/src/main/java/com/azure/data/appconfiguration/ConfigurationAsyncClient.java

@@ -9,7 +9,6 @@
 import com.azure.core.exception.HttpResponseException;
 import com.azure.core.exception.ResourceModifiedException;
 import com.azure.core.exception.ResourceNotFoundException;
-import com.azure.core.experimental.models.PollResult;
 import com.azure.core.http.HttpResponse;
 import com.azure.core.http.MatchConditions;
 import com.azure.core.http.rest.PagedFlux;
@@ -18,6 +17,7 @@
 import com.azure.core.http.rest.SimpleResponse;
 import com.azure.core.util.Context;
 import com.azure.core.util.logging.ClientLogger;
+import com.azure.core.util.polling.PollOperationDetails;
 import com.azure.core.util.polling.PollerFlux;
 import com.azure.data.appconfiguration.implementation.AzureAppConfigurationImpl;
 import com.azure.data.appconfiguration.implementation.CreateSnapshotUtilClient;
@@ -1218,7 +1218,7 @@ public PagedFlux<ConfigurationSetting> listRevisions(SettingSelector selector) {
      * has failed. The completed operation returns a {@link ConfigurationSnapshot}.
      */
     @ServiceMethod(returns = ReturnType.LONG_RUNNING_OPERATION)
-    public PollerFlux<PollResult, ConfigurationSnapshot> beginCreateSnapshot(
+    public PollerFlux<PollOperationDetails, ConfigurationSnapshot> beginCreateSnapshot(
         String snapshotName, ConfigurationSnapshot snapshot) {
         return createSnapshotUtilClient.beginCreateSnapshot(snapshotName, snapshot);
     }

sdk/appconfiguration/azure-data-appconfiguration/src/main/java/com/azure/data/appconfiguration/ConfigurationClient.java

@@ -9,7 +9,6 @@
 import com.azure.core.exception.HttpResponseException;
 import com.azure.core.exception.ResourceModifiedException;
 import com.azure.core.exception.ResourceNotFoundException;
-import com.azure.core.experimental.models.PollResult;
 import com.azure.core.http.HttpResponse;
 import com.azure.core.http.MatchConditions;
 import com.azure.core.http.rest.PagedIterable;
@@ -19,6 +18,7 @@
 import com.azure.core.http.rest.SimpleResponse;
 import com.azure.core.util.Context;
 import com.azure.core.util.logging.ClientLogger;
+import com.azure.core.util.polling.PollOperationDetails;
 import com.azure.core.util.polling.SyncPoller;
 import com.azure.data.appconfiguration.implementation.AzureAppConfigurationImpl;
 import com.azure.data.appconfiguration.implementation.CreateSnapshotUtilClient;
@@ -1252,7 +1252,7 @@ public PagedIterable<ConfigurationSetting> listRevisions(SettingSelector selecto
      * String snapshotName = &quot;&#123;snapshotName&#125;&quot;;
      * Context ctx = new Context&#40;key2, value2&#41;;
      *
-     * SyncPoller&lt;PollResult, ConfigurationSnapshot&gt; poller =
+     * SyncPoller&lt;PollOperationDetails, ConfigurationSnapshot&gt; poller =
      *     client.beginCreateSnapshot&#40;snapshotName,
      *         new ConfigurationSnapshot&#40;filters&#41;.setRetentionPeriod&#40;Duration.ofHours&#40;1&#41;&#41;, ctx&#41;;
      * poller.setPollInterval&#40;Duration.ofSeconds&#40;10&#41;&#41;;
@@ -1270,7 +1270,7 @@ public PagedIterable<ConfigurationSetting> listRevisions(SettingSelector selecto
      * has failed. The completed operation returns a {@link ConfigurationSnapshot}.
      */
     @ServiceMethod(returns = ReturnType.LONG_RUNNING_OPERATION)
-    public SyncPoller<PollResult, ConfigurationSnapshot> beginCreateSnapshot(
+    public SyncPoller<PollOperationDetails, ConfigurationSnapshot> beginCreateSnapshot(
         String snapshotName, ConfigurationSnapshot snapshot, Context context) {
         return createSnapshotUtilClient.beginCreateSnapshot(snapshotName, snapshot, context);
     }

sdk/appconfiguration/azure-data-appconfiguration/src/main/java/com/azure/data/appconfiguration/ConfigurationServiceVersion.java

@@ -15,9 +15,9 @@ public enum ConfigurationServiceVersion implements ServiceVersion {
     V1_0("1.0"),
 
     /**
-     * Service version {@code 2022-11-01-preview}.
+     * Service version {@code 2023-10-01}.
      */
-    V2022_11_01_PREVIEW("2022-11-01-preview");
+    V2023_10_01("2023-10-01");
 
     private final String version;
 
@@ -39,6 +39,6 @@ public String getVersion() {
      * @return the latest {@link ConfigurationServiceVersion}
      */
     public static ConfigurationServiceVersion getLatest() {
-        return V2022_11_01_PREVIEW;
+        return V2023_10_01;
     }
 }

sdk/appconfiguration/azure-data-appconfiguration/src/main/java/com/azure/data/appconfiguration/implementation/CreateSnapshotUtilClient.java

@@ -4,13 +4,13 @@
 package com.azure.data.appconfiguration.implementation;
 
 
-import com.azure.core.experimental.models.PollResult;
 import com.azure.core.http.rest.Response;
 import com.azure.core.http.rest.ResponseBase;
 import com.azure.core.util.BinaryData;
 import com.azure.core.util.Context;
 import com.azure.core.util.logging.ClientLogger;
 import com.azure.core.util.polling.LongRunningOperationStatus;
+import com.azure.core.util.polling.PollOperationDetails;
 import com.azure.core.util.polling.PollResponse;
 import com.azure.core.util.polling.PollerFlux;
 import com.azure.core.util.polling.PollingContext;
@@ -44,7 +44,7 @@ public CreateSnapshotUtilClient(AzureAppConfigurationImpl service) {
         this.service = service;
     }
 
-    public PollerFlux<PollResult, ConfigurationSnapshot> beginCreateSnapshot(String name,
+    public PollerFlux<PollOperationDetails, ConfigurationSnapshot> beginCreateSnapshot(String name,
         ConfigurationSnapshot snapshot) {
         try {
             return new PollerFlux<>(
@@ -54,7 +54,7 @@ public PollerFlux<PollResult, ConfigurationSnapshot> beginCreateSnapshot(String
                         .map(response -> {
                             final Map<String, String> pollResponse = new HashMap<>();
                             pollResponse.put("id", response.getDeserializedHeaders().getOperationLocation());
-                            return BinaryData.fromObject(pollResponse).toObject(PollResult.class);
+                            return BinaryData.fromObject(pollResponse).toObject(PollOperationDetails.class);
                         })),
                 pollingOperation(
                     operationId -> service.getOperationDetailsWithResponseAsync(name, Context.NONE)),
@@ -70,7 +70,7 @@ public PollerFlux<PollResult, ConfigurationSnapshot> beginCreateSnapshot(String
         }
     }
 
-    public SyncPoller<PollResult, ConfigurationSnapshot> beginCreateSnapshot(String name,
+    public SyncPoller<PollOperationDetails, ConfigurationSnapshot> beginCreateSnapshot(String name,
         ConfigurationSnapshot snapshot, Context context) {
         try {
             final Context finalContext = getNotNullContext(context);
@@ -93,8 +93,8 @@ public SyncPoller<PollResult, ConfigurationSnapshot> beginCreateSnapshot(String
     }
 
     // Activation operation
-    private Function<PollingContext<PollResult>, Mono<PollResult>>
-        activationOperation(Mono<PollResult> operationResult) {
+    private Function<PollingContext<PollOperationDetails>, Mono<PollOperationDetails>>
+        activationOperation(Mono<PollOperationDetails> operationResult) {
         return pollingContext -> {
             try {
                 return operationResult;
@@ -104,7 +104,7 @@ public SyncPoller<PollResult, ConfigurationSnapshot> beginCreateSnapshot(String
         };
     }
 
-    private Function<PollingContext<PollResult>, PollResult>
+    private Function<PollingContext<PollOperationDetails>, PollOperationDetails>
         activationOperationSync(String name, ConfigurationSnapshot snapshot, Context context) {
         return pollingContext -> {
             try {
@@ -113,19 +113,19 @@ public SyncPoller<PollResult, ConfigurationSnapshot> beginCreateSnapshot(String
                     service.createSnapshotWithResponse(name, snapshot, finalContext);
                 final Map<String, String> pollResponse = new HashMap<>();
                 pollResponse.put("id", snapshotWithResponse.getDeserializedHeaders().getOperationLocation());
-                return BinaryData.fromObject(pollResponse).toObject(PollResult.class);
+                return BinaryData.fromObject(pollResponse).toObject(PollOperationDetails.class);
             } catch (RuntimeException ex) {
                 throw LOGGER.logExceptionAsError(ex);
             }
         };
     }
 
     // Polling operation
-    private Function<PollingContext<PollResult>, Mono<PollResponse<PollResult>>>
+    private Function<PollingContext<PollOperationDetails>, Mono<PollResponse<PollOperationDetails>>>
         pollingOperation(Function<String, Mono<Response<OperationDetails>>> pollingFunction) {
         return pollingContext -> {
             try {
-                final PollResponse<PollResult> pollResponse =
+                final PollResponse<PollOperationDetails> pollResponse =
                     pollingContext.getLatestResponse();
                 final String operationId = pollResponse.getValue().getOperationId();
                 return pollingFunction.apply(operationId)
@@ -137,11 +137,11 @@ public SyncPoller<PollResult, ConfigurationSnapshot> beginCreateSnapshot(String
         };
     }
 
-    private Function<PollingContext<PollResult>, PollResponse<PollResult>>
+    private Function<PollingContext<PollOperationDetails>, PollResponse<PollOperationDetails>>
         pollingOperationSync(Function<String, Response<OperationDetails>> pollingFunction) {
         return pollingContext -> {
             try {
-                final PollResponse<PollResult> pollResponse =
+                final PollResponse<PollOperationDetails> pollResponse =
                     pollingContext.getLatestResponse();
                 return processResponse(pollingFunction.apply(pollResponse.getValue().getOperationId()), pollResponse);
             } catch (RuntimeException ex) {
@@ -151,7 +151,7 @@ public SyncPoller<PollResult, ConfigurationSnapshot> beginCreateSnapshot(String
     }
 
     // Fetching operation
-    private Function<PollingContext<PollResult>,
+    private Function<PollingContext<PollOperationDetails>,
                         Mono<ConfigurationSnapshot>> fetchingOperation(
         Function<String, Mono<ConfigurationSnapshot>> fetchingFunction) {
         return pollingContext -> {
@@ -164,7 +164,7 @@ Mono<ConfigurationSnapshot>> fetchingOperation(
         };
     }
 
-    private Function<PollingContext<PollResult>, ConfigurationSnapshot> fetchingOperationSync(
+    private Function<PollingContext<PollOperationDetails>, ConfigurationSnapshot> fetchingOperationSync(
         Function<String, ConfigurationSnapshot> fetchingFunction) {
         return pollingContext -> {
             try {
@@ -176,9 +176,9 @@ private Function<PollingContext<PollResult>, ConfigurationSnapshot> fetchingOper
         };
     }
 
-    private PollResponse<PollResult> processResponse(
+    private PollResponse<PollOperationDetails> processResponse(
         Response<OperationDetails> response,
-        PollResponse<PollResult> operationResultPollResponse) {
+        PollResponse<PollOperationDetails> operationResultPollResponse) {
         LongRunningOperationStatus status;
         State state = response.getValue().getStatus();
         if (NOT_STARTED.equals(state) || RUNNING.equals(state)) {