Make changes on public APIs direct under 'search' (#8393)

* Address PR feedback on search package.

Author: sima-zhu

sdk/search/azure-search/src/main/java/com/azure/search/DataSources.java

@@ -1,5 +1,6 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License.
+
 package com.azure.search;
 
 import java.util.HashMap;
@@ -12,7 +13,7 @@
  * <p>
  * If the schema is known, user can convert the properties to a specific object type
  */
-public class Document extends HashMap<String, Object> {
+public final class Document extends HashMap<String, Object> {
     private static final long serialVersionUID = 1L;
 
     /**
@@ -25,9 +26,9 @@ public Document() {
     /**
      * Constructs a new {@code Document} with the same mappings as the specified {@code Map}.
      *
-     * @param m the map whose mappings are to be placed in this map
+     * @param propertyMap the map whose mappings are to be placed in this map
      */
-    public Document(Map<? extends String, ?> m) {
-        super(m);
+    public Document(Map<? extends String, ?> propertyMap) {
+        super(propertyMap);
     }
 }

sdk/search/azure-search/src/main/java/com/azure/search/Document.java

@@ -14,7 +14,7 @@
  * An {@code IndexBatchException} is thrown whenever Azure Cognitive Search index call was only partially successful.
  * Users can inspect the indexingResults to determine the operation(s) that have failed.
  */
-public class IndexBatchException extends AzureException {
+public final class IndexBatchException extends AzureException {
     private static final long serialVersionUID = -3478124828996650248L;
     private static final String MESSAGE_FORMAT = "%s of %s indexing actions in the batch failed. The remaining"
         + " actions succeeded and modified the index. Check indexingResults for the status of each index action.";

sdk/search/azure-search/src/main/java/com/azure/search/IndexBatchException.java

@@ -10,7 +10,7 @@
  * @see SearchIndexClientBuilder
  * @see SearchServiceClientBuilder
  */
-public class SearchApiKeyCredential {
+public final class SearchApiKeyCredential {
     private final String apiKey;
 
     /**

sdk/search/azure-search/src/main/java/com/azure/search/SearchApiKeyCredential.java

@@ -46,7 +46,7 @@
  * Cognitive Search Asynchronous Client to query an index and upload, merge, or delete documents
  */
 @ServiceClient(builder = SearchIndexClientBuilder.class, isAsync = true)
-public class SearchIndexAsyncClient {
+public final class SearchIndexAsyncClient {
 
     /*
      * Representation of the Multi-Status HTTP response code.
@@ -61,7 +61,7 @@ public class SearchIndexAsyncClient {
     /**
      * Search REST API Version
      */
-    private final SearchServiceVersion apiVersion;
+    private final SearchServiceVersion searchServiceVersion;
 
     /**
      * The endpoint for the Azure Cognitive Search service.
@@ -91,31 +91,31 @@ public class SearchIndexAsyncClient {
     /**
      * Package private constructor to be used by {@link SearchIndexClientBuilder}
      */
-    SearchIndexAsyncClient(String endpoint, String indexName, SearchServiceVersion apiVersion,
-        HttpPipeline httpPipeline) {
+    SearchIndexAsyncClient(String endpoint, String indexName, SearchServiceVersion searchServiceVersion,
+                           HttpPipeline httpPipeline) {
 
         SearchServiceUrlParts parts = SearchServiceUrlParser.parseServiceUrlParts(endpoint);
 
         if (CoreUtils.isNullOrEmpty(indexName)) {
             throw logger.logExceptionAsError(new NullPointerException("Invalid indexName"));
         }
-        if (apiVersion == null) {
-            throw logger.logExceptionAsError(new NullPointerException("Invalid apiVersion"));
+        if (searchServiceVersion == null) {
+            throw logger.logExceptionAsError(new NullPointerException("Invalid search service version"));
         }
         if (httpPipeline == null) {
             throw logger.logExceptionAsError(new NullPointerException("Invalid httpPipeline"));
         }
 
         this.endpoint = endpoint;
         this.indexName = indexName;
-        this.apiVersion = apiVersion;
+        this.searchServiceVersion = searchServiceVersion;
         this.httpPipeline = httpPipeline;
 
         restClient = new SearchIndexRestClientBuilder()
             .searchServiceName(parts.serviceName)
             .indexName(indexName)
             .searchDnsSuffix(parts.dnsSuffix)
-            .apiVersion(apiVersion.getVersion())
+            .apiVersion(searchServiceVersion.getVersion())
             .pipeline(httpPipeline)
             .serializer(SERIALIZER)
             .build();
@@ -342,8 +342,8 @@ Mono<Response<IndexDocumentsResult>> deleteDocumentsWithResponse(Iterable<?> doc
      *
      * @return The version of the Search service the client is using.
      */
-    public SearchServiceVersion getApiVersion() {
-        return this.apiVersion;
+    public SearchServiceVersion getServiceVersion() {
+        return this.searchServiceVersion;
     }
 
     /**

sdk/search/azure-search/src/main/java/com/azure/search/SearchIndexAsyncClient.java

@@ -25,7 +25,7 @@
  * Cognitive Search Synchronous Client to query an index and upload, merge, or delete documents
  */
 @ServiceClient(builder = SearchIndexClientBuilder.class)
-public class SearchIndexClient {
+public final class SearchIndexClient {
 
     private final SearchIndexAsyncClient asyncClient;
 
@@ -231,8 +231,8 @@ public Response<IndexDocumentsResult> deleteDocumentsWithResponse(Iterable<?> do
      *
      * @return The version of the Search service the client is using.
      */
-    public SearchServiceVersion getApiVersion() {
-        return asyncClient.getApiVersion();
+    public SearchServiceVersion getServiceVersion() {
+        return asyncClient.getServiceVersion();
     }
 
     /**

sdk/search/azure-search/src/main/java/com/azure/search/SearchIndexClient.java

@@ -39,7 +39,7 @@
  * {@code .endpoint()} the index name through {@code .indexName()} the API key through {@code .credential()}
  */
 @ServiceClientBuilder(serviceClients = {SearchIndexClient.class, SearchIndexAsyncClient.class})
-public class SearchIndexClientBuilder {
+public final class SearchIndexClientBuilder {
 
     // This header tells the server to return the request id in the HTTP response. Useful for correlation with what
     // request was sent.
@@ -50,7 +50,7 @@ public class SearchIndexClientBuilder {
     private static final String VERSION = "version";
 
     SearchApiKeyCredential searchApiKeyCredential;
-    SearchServiceVersion apiVersion;
+    SearchServiceVersion searchServiceVersion;
     String endpoint;
     HttpClient httpClient;
     HttpLogOptions httpLogOptions;
@@ -68,7 +68,7 @@ public class SearchIndexClientBuilder {
      * Default Constructor
      */
     public SearchIndexClientBuilder() {
-        apiVersion = SearchServiceVersion.getLatest();
+        searchServiceVersion = SearchServiceVersion.getLatest();
         policies = new ArrayList<>();
         httpClient = HttpClient.createDefault();
         httpLogOptions = new HttpLogOptions();
@@ -83,14 +83,14 @@ public SearchIndexClientBuilder() {
     /**
      * Sets the api version to work against
      *
-     * @param apiVersion api version
+     * @param searchServiceVersion api version
      * @return the updated SearchIndexClientBuilder object
      */
-    public SearchIndexClientBuilder apiVersion(SearchServiceVersion apiVersion) {
-        if (apiVersion == null) {
-            throw logger.logExceptionAsError(new IllegalArgumentException("Invalid apiVersion"));
+    public SearchIndexClientBuilder searchServiceVersion(SearchServiceVersion searchServiceVersion) {
+        if (searchServiceVersion == null) {
+            throw logger.logExceptionAsError(new IllegalArgumentException("Invalid searchServiceVersion"));
         }
-        this.apiVersion = apiVersion;
+        this.searchServiceVersion = searchServiceVersion;
         return this;
     }
 
@@ -236,7 +236,7 @@ public SearchIndexAsyncClient buildAsyncClient() {
         policies.add(retryPolicy == null ? new RetryPolicy() : retryPolicy);
         HttpPolicyProviders.addAfterRetryPolicies(policies);
 
-        return new SearchIndexAsyncClient(endpoint, indexName, apiVersion, prepareForBuildClient());
+        return new SearchIndexAsyncClient(endpoint, indexName, searchServiceVersion, prepareForBuildClient());
     }
 
     HttpPipeline prepareForBuildClient() {

sdk/search/azure-search/src/main/java/com/azure/search/SearchIndexClientBuilder.java

@@ -43,7 +43,7 @@ public final class SearchServiceAsyncClient {
     /**
      * Search REST API Version
      */
-    private final SearchServiceVersion apiVersion;
+    private final SearchServiceVersion searchServiceVersion;
 
     /**
      * The endpoint for the Azure Cognitive Search service.
@@ -65,25 +65,25 @@ public final class SearchServiceAsyncClient {
      */
     private final HttpPipeline httpPipeline;
 
-    SearchServiceAsyncClient(String endpoint, SearchServiceVersion apiVersion, HttpPipeline httpPipeline) {
+    SearchServiceAsyncClient(String endpoint, SearchServiceVersion searchServiceVersion, HttpPipeline httpPipeline) {
 
         SearchServiceUrlParts parts = SearchServiceUrlParser.parseServiceUrlParts(endpoint);
 
-        if (apiVersion == null) {
+        if (searchServiceVersion == null) {
             throw logger.logExceptionAsError(new NullPointerException("Invalid apiVersion"));
         }
         if (httpPipeline == null) {
             throw logger.logExceptionAsError(new NullPointerException("Invalid httpPipeline"));
         }
 
         this.endpoint = endpoint;
-        this.apiVersion = apiVersion;
+        this.searchServiceVersion = searchServiceVersion;
         this.httpPipeline = httpPipeline;
 
         this.restClient = new SearchServiceRestClientBuilder()
             .searchServiceName(parts.serviceName)
             .searchDnsSuffix(parts.dnsSuffix)
-            .apiVersion(apiVersion.getVersion())
+            .apiVersion(searchServiceVersion.getVersion())
             .pipeline(httpPipeline)
             .build();
     }
@@ -105,16 +105,16 @@ HttpPipeline getHttpPipeline() {
      * @return a {@link SearchIndexAsyncClient} created from the service client configuration
      */
     public SearchIndexAsyncClient getIndexClient(String indexName) {
-        return new SearchIndexAsyncClient(endpoint, indexName, apiVersion, httpPipeline);
+        return new SearchIndexAsyncClient(endpoint, indexName, searchServiceVersion, httpPipeline);
     }
 
     /**
-     * Gets Client Api Version.
+     * Gets search service version.
      *
-     * @return the apiVersion value.
+     * @return the search service version value.
      */
-    public SearchServiceVersion getApiVersion() {
-        return this.apiVersion;
+    public SearchServiceVersion getServiceVersion() {
+        return this.searchServiceVersion;
     }
 
     /**

sdk/search/azure-search/src/main/java/com/azure/search/SearchServiceAsyncClient.java

@@ -52,12 +52,12 @@ public SearchIndexClient getIndexClient(String indexName) {
     }
 
     /**
-     * Gets Client Api Version.
+     * Gets search service version.
      *
-     * @return the apiVersion value.
+     * @return the search service version value.
      */
-    public SearchServiceVersion getApiVersion() {
-        return this.asyncClient.getApiVersion();
+    public SearchServiceVersion getServiceVersion() {
+        return this.asyncClient.getServiceVersion();
     }
 
     /**

sdk/search/azure-search/src/main/java/com/azure/search/SearchServiceClient.java

@@ -32,15 +32,15 @@
  * {@code .endpoint()} the API key through {@code .credential()}
  */
 @ServiceClientBuilder(serviceClients = {SearchServiceClient.class, SearchServiceAsyncClient.class})
-public class SearchServiceClientBuilder {
+public final class SearchServiceClientBuilder {
     private final ClientLogger logger = new ClientLogger(SearchServiceClientBuilder.class);
 
     private static final String SEARCH_PROPERTIES = "azure-search.properties";
     private static final String NAME = "name";
     private static final String VERSION = "version";
 
     SearchApiKeyCredential searchApiKeyCredential;
-    SearchServiceVersion apiVersion;
+    SearchServiceVersion searchServiceVersion;
     String endpoint;
     HttpClient httpClient;
     HttpLogOptions httpLogOptions;
@@ -53,7 +53,7 @@ public class SearchServiceClientBuilder {
      * Default Constructor
      */
     public SearchServiceClientBuilder() {
-        apiVersion = SearchServiceVersion.getLatest();
+        searchServiceVersion = SearchServiceVersion.getLatest();
         policies = new ArrayList<>();
         httpClient = HttpClient.createDefault();
         httpLogOptions = new HttpLogOptions();
@@ -66,14 +66,14 @@ public SearchServiceClientBuilder() {
     /**
      * Sets the api version to work against
      *
-     * @param apiVersion api version
+     * @param searchServiceVersion api version
      * @return the updated SearchServiceClientBuilder object
      */
-    public SearchServiceClientBuilder apiVersion(SearchServiceVersion apiVersion) {
-        if (apiVersion == null) {
-            throw logger.logExceptionAsError(new IllegalArgumentException("Invalid apiVersion"));
+    public SearchServiceClientBuilder searchServiceVersion(SearchServiceVersion searchServiceVersion) {
+        if (searchServiceVersion == null) {
+            throw logger.logExceptionAsError(new IllegalArgumentException("Invalid searchServiceVersion"));
         }
-        this.apiVersion = apiVersion;
+        this.searchServiceVersion = searchServiceVersion;
         return this;
     }
 
@@ -169,7 +169,7 @@ public SearchServiceClient buildClient() {
      * @return a {@link SearchIndexAsyncClient} created from the configurations in this builder.
      */
     public SearchServiceAsyncClient buildAsyncClient() {
-        return new SearchServiceAsyncClient(endpoint, apiVersion, prepareForBuildClient());
+        return new SearchServiceAsyncClient(endpoint, searchServiceVersion, prepareForBuildClient());
     }
 
     HttpPipeline prepareForBuildClient() {