Enable array open v2 and query v3 by default. (#5207)

[[sc-47320](https://app.shortcut.com/tiledb-inc/story/47320)]

This PR just changes the two configuration variables that toggle "Array
open v2" and "Query v3" features to ON by default.

---
TYPE: FEATURE
DESC: Enable array open v2 and query v3 by default.

Author: ypatia

test/src/unit-capi-config.cc

@@ -239,8 +239,8 @@ void check_save_to_file() {
   ss << "rest.retry_initial_delay_ms 500\n";
   ss << "rest.server_address https://api.tiledb.com\n";
   ss << "rest.server_serialization_format CAPNP\n";
-  ss << "rest.use_refactored_array_open false\n";
-  ss << "rest.use_refactored_array_open_and_query_submit false\n";
+  ss << "rest.use_refactored_array_open true\n";
+  ss << "rest.use_refactored_array_open_and_query_submit true\n";
   ss << "sm.allow_separate_attribute_writes false\n";
   ss << "sm.allow_updates_experimental false\n";
   ss << "sm.check_coord_dups true\n";
@@ -579,13 +579,13 @@ TEST_CASE("C API: Test config iter", "[capi][config]") {
   CHECK(rc == TILEDB_OK);
   CHECK(error == nullptr);
   rc = tiledb_config_set(
-      config, "rest.use_refactored_array_open", "true", &error);
+      config, "rest.use_refactored_array_open", "false", &error);
   CHECK(rc == TILEDB_OK);
   CHECK(error == nullptr);
   rc = tiledb_config_set(
       config,
       "rest.use_refactored_array_open_and_query_submit",
-      "true",
+      "false",
       &error);
   CHECK(rc == TILEDB_OK);
   CHECK(error == nullptr);
@@ -613,8 +613,8 @@ TEST_CASE("C API: Test config iter", "[capi][config]") {
   all_param_values["rest.load_metadata_on_array_open"] = "false";
   all_param_values["rest.load_non_empty_domain_on_array_open"] = "false";
   all_param_values["rest.load_enumerations_on_array_open"] = "false";
-  all_param_values["rest.use_refactored_array_open"] = "true";
-  all_param_values["rest.use_refactored_array_open_and_query_submit"] = "true";
+  all_param_values["rest.use_refactored_array_open"] = "false";
+  all_param_values["rest.use_refactored_array_open_and_query_submit"] = "false";
   all_param_values["rest.payer_namespace"] = "";
   all_param_values["sm.allow_separate_attribute_writes"] = "false";
   all_param_values["sm.allow_updates_experimental"] = "false";

tiledb/api/c_api/config/config_api_external.h

@@ -741,13 +741,13 @@ TILEDB_EXPORT void tiledb_config_free(tiledb_config_t** config) TILEDB_NOEXCEPT;
  *    with the open array <br>
  *    **Default**: true
  * - `rest.use_refactored_array_open` <br>
- *    If true, the new, experimental REST routes and APIs for opening an array
+ *    If true, the new REST routes and APIs for opening an array
  *    will be used <br>
- *    **Default**: false
+ *    **Default**: true
  * - `rest.use_refactored_array_open_and_query_submit` <br>
- *    If true, the new, experimental REST routes and APIs for opening an array
- *    and submitting a query will be used <br>
- *    **Default**: false
+ *    If true, the new REST routes and APIs for opening an array and submitting
+ * a query will be used <br>
+ *    **Default**: true
  * - `rest.curl.buffer_size` <br>
  *    Set curl buffer size for REST requests <br>
  *    **Default**: 524288 (512KB)

tiledb/sm/config/config.cc

@@ -94,8 +94,8 @@ const std::string Config::REST_CURL_VERBOSE = "false";
 const std::string Config::REST_LOAD_ENUMERATIONS_ON_ARRAY_OPEN = "true";
 const std::string Config::REST_LOAD_METADATA_ON_ARRAY_OPEN = "true";
 const std::string Config::REST_LOAD_NON_EMPTY_DOMAIN_ON_ARRAY_OPEN = "true";
-const std::string Config::REST_USE_REFACTORED_ARRAY_OPEN = "false";
-const std::string Config::REST_USE_REFACTORED_QUERY_SUBMIT = "false";
+const std::string Config::REST_USE_REFACTORED_ARRAY_OPEN = "true";
+const std::string Config::REST_USE_REFACTORED_QUERY_SUBMIT = "true";
 const std::string Config::REST_PAYER_NAMESPACE = "";
 const std::string Config::SM_ALLOW_SEPARATE_ATTRIBUTE_WRITES = "false";
 const std::string Config::SM_ALLOW_UPDATES_EXPERIMENTAL = "false";

tiledb/sm/cpp_api/config.h

@@ -916,13 +916,13 @@ class Config {
    *    together with the open array <br>
    *    **Default**: true
    * - `rest.use_refactored_array_open` <br>
-   *    If true, the new, experimental REST routes and APIs for opening an array
-   *    will be used <br>
-   *    **Default**: false
+   *    If true, the new REST routes and APIs for opening an array will be used
+   *    <br>
+   *    **Default**: true
    * - `rest.use_refactored_array_open_and_query_submit` <br>
-   *    If true, the new, experimental REST routes and APIs for opening an array
-   *    and submitting a query will be used <br>
-   *    **Default**: false
+   *    If true, the new REST routes and APIs for opening an array and
+   *    submitting a query will be used <br>
+   *    **Default**: true
    * - `rest.curl.buffer_size` <br>
    *    Set curl buffer size for REST requests <br>
    *    **Default**: 524288 (512KB)