This is an automated email from the ASF dual-hosted git repository.

alsuliman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git


The following commit(s) were added to refs/heads/master by this push:
     new 57aefaa657 [ASTERIXDB-3511][COMP] Set 'ordered field' to true as 
default configuration
57aefaa657 is described below

commit 57aefaa657e78eed014aa2aa5b192490601dd4ca
Author: AnkitPrabhu <[email protected]>
AuthorDate: Tue Oct 15 10:12:16 2024 +0530

    [ASTERIXDB-3511][COMP] Set 'ordered field' to true as default configuration
    
    - user model changes: no
    - storage format changes: no
    - interface changes: no
    
    Ext-ref: MB-61971, MB-64515
    
    Change-Id: I03b39e3205200c0fdb21ead864878e720718a6c1
    Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18964
    Integration-Tests: Jenkins <[email protected]>
    Reviewed-by: Ali Alsuliman <[email protected]>
    Tested-by: Ali Alsuliman <[email protected]>
---
 .../apache/asterix/compiler/provider/SqlppCompilationProvider.java  | 2 +-
 .../src/main/java/org/apache/asterix/api/common/APIFramework.java   | 4 ++--
 asterixdb/asterix-app/src/main/resources/cc.conf                    | 1 +
 asterixdb/asterix-app/src/main/resources/cc2.conf                   | 1 +
 asterixdb/asterix-app/src/main/resources/cc3.conf                   | 1 +
 asterixdb/asterix-app/src/main/resources/cc4.conf                   | 1 +
 asterixdb/asterix-app/src/main/resources/cc_no_cbo.conf             | 1 +
 .../asterix-app/src/main/resources/cc_static_partitioning.conf      | 1 +
 .../apache/asterix/api/common/AsterixHyracksIntegrationUtil.java    | 3 ++-
 .../org/apache/asterix/api/common/CloudStorageIntegrationUtil.java  | 2 +-
 .../src/test/java/org/apache/asterix/test/common/TestExecutor.java  | 2 +-
 asterixdb/asterix-app/src/test/resources/cc-analyze.conf            | 3 ++-
 asterixdb/asterix-app/src/test/resources/cc-cbotest.conf            | 1 +
 .../asterix-app/src/test/resources/cc-cloud-storage-azblob.conf     | 1 +
 asterixdb/asterix-app/src/test/resources/cc-cloud-storage-gcs.conf  | 1 +
 .../resources/{cc-cloud-storage.conf => cc-cloud-storage-main.conf} | 2 +-
 asterixdb/asterix-app/src/test/resources/cc-cloud-storage.conf      | 1 +
 asterixdb/asterix-app/src/test/resources/cc-columnar.conf           | 1 +
 asterixdb/asterix-app/src/test/resources/{cc.conf => cc-main.conf}  | 2 +-
 asterixdb/asterix-app/src/test/resources/cc-single-lookup.conf      | 1 +
 asterixdb/asterix-app/src/test/resources/cc-single-partition.conf   | 1 +
 asterixdb/asterix-app/src/test/resources/cc-single.conf             | 1 +
 .../asterix-app/src/test/resources/cc-small-txn-log-partition.conf  | 1 +
 asterixdb/asterix-app/src/test/resources/cc-ssl.conf                | 1 +
 asterixdb/asterix-app/src/test/resources/cc-storage.conf            | 1 +
 asterixdb/asterix-app/src/test/resources/cc.conf                    | 1 +
 .../results/api/cluster_state_1/cluster_state_1.1.regexadm          | 2 +-
 .../api/cluster_state_1_full/cluster_state_1_full.1.regexadm        | 2 +-
 .../api/cluster_state_1_less/cluster_state_1_less.1.regexadm        | 2 +-
 .../java/org/apache/asterix/common/config/CompilerProperties.java   | 6 +++---
 .../java/org/apache/asterix/common/config/OptimizationConfUtil.java | 2 +-
 .../om/typecomputer/impl/OpenRecordConstructorResultType.java       | 4 ++--
 .../asterix-server/src/test/resources/NCServiceExecutionIT/cc.conf  | 3 ++-
 asterixdb/asterix-server/src/test/resources/NcLifecycleIT/cc.conf   | 1 +
 asterixdb/asterix-server/src/test/resources/ReplicationIT/cc.conf   | 1 +
 .../org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java | 2 +-
 .../algebricks/core/rewriter/base/PhysicalOptimizationConfig.java   | 2 +-
 37 files changed, 44 insertions(+), 21 deletions(-)

diff --git 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
index eef5884031..10574b1cb2 100644
--- 
a/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
+++ 
b/asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
@@ -90,7 +90,7 @@ public class SqlppCompilationProvider implements 
ILangCompilationProvider {
                 CompilerProperties.COMPILER_SORT_SAMPLES_KEY, 
CompilerProperties.COMPILER_EXTERNALSCANMEMORY_KEY,
                 CompilerProperties.COMPILER_INDEXONLY_KEY, 
CompilerProperties.COMPILER_INTERNAL_SANITYCHECK_KEY,
                 CompilerProperties.COMPILER_EXTERNAL_FIELD_PUSHDOWN_KEY, 
CompilerProperties.COMPILER_SUBPLAN_MERGE_KEY,
-                CompilerProperties.COMPILER_SUBPLAN_NESTEDPUSHDOWN_KEY, 
CompilerProperties.COMPILER_ORDERFIELDS_KEY,
+                CompilerProperties.COMPILER_SUBPLAN_NESTEDPUSHDOWN_KEY, 
CompilerProperties.COMPILER_ORDERED_FIELDS_KEY,
                 CompilerProperties.COMPILER_ARRAYINDEX_KEY, 
CompilerProperties.COMPILER_CBO_KEY,
                 CompilerProperties.COMPILER_CBO_TEST_KEY, 
CompilerProperties.COMPILER_FORCE_JOIN_ORDER_KEY,
                 CompilerProperties.COMPILER_QUERY_PLAN_SHAPE_KEY, 
CompilerProperties.COMPILER_MIN_MEMORY_ALLOCATION_KEY,
diff --git 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
index db7c21a558..be36225e54 100644
--- 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
+++ 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
@@ -238,8 +238,8 @@ public class APIFramework {
         Map<String, Object> querySpecificConfig = validateConfig(config, 
sourceLoc);
         final PhysicalOptimizationConfig physOptConf = 
OptimizationConfUtil.createPhysicalOptimizationConf(
                 compilerProperties, querySpecificConfig, 
configurableParameterNames, sourceLoc);
-        if (!config.containsKey(CompilerProperties.COMPILER_ORDERFIELDS_KEY)) {
-            config.put(CompilerProperties.COMPILER_ORDERFIELDS_KEY, 
Boolean.toString(physOptConf.isOrderField()));
+        if 
(!config.containsKey(CompilerProperties.COMPILER_ORDERED_FIELDS_KEY)) {
+            config.put(CompilerProperties.COMPILER_ORDERED_FIELDS_KEY, 
Boolean.toString(physOptConf.isOrderField()));
         }
 
         boolean cboMode = physOptConf.getCBOMode() || 
physOptConf.getCBOTestMode();
diff --git a/asterixdb/asterix-app/src/main/resources/cc.conf 
b/asterixdb/asterix-app/src/main/resources/cc.conf
index e0eb88d595..f5a496c95b 100644
--- a/asterixdb/asterix-app/src/main/resources/cc.conf
+++ b/asterixdb/asterix-app/src/main/resources/cc.conf
@@ -59,6 +59,7 @@ compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
 compiler.sort.parallel=false
 compiler.internal.sanitycheck=true
+compiler.ordered.fields=false
 messaging.frame.size=4096
 messaging.frame.count=512
 storage.buffercache.pagesize=32KB
diff --git a/asterixdb/asterix-app/src/main/resources/cc2.conf 
b/asterixdb/asterix-app/src/main/resources/cc2.conf
index 8868a5423a..bb2e4575e9 100644
--- a/asterixdb/asterix-app/src/main/resources/cc2.conf
+++ b/asterixdb/asterix-app/src/main/resources/cc2.conf
@@ -53,6 +53,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 compiler.parallelism=-1
 messaging.frame.size=4096
 messaging.frame.count=512
diff --git a/asterixdb/asterix-app/src/main/resources/cc3.conf 
b/asterixdb/asterix-app/src/main/resources/cc3.conf
index 71ad17b62c..cc83da0ee0 100644
--- a/asterixdb/asterix-app/src/main/resources/cc3.conf
+++ b/asterixdb/asterix-app/src/main/resources/cc3.conf
@@ -53,6 +53,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 compiler.parallelism=3
 messaging.frame.size=4096
 messaging.frame.count=512
diff --git a/asterixdb/asterix-app/src/main/resources/cc4.conf 
b/asterixdb/asterix-app/src/main/resources/cc4.conf
index bf37499874..0d5b9adf44 100644
--- a/asterixdb/asterix-app/src/main/resources/cc4.conf
+++ b/asterixdb/asterix-app/src/main/resources/cc4.conf
@@ -50,6 +50,7 @@ compiler.sortmemory=320KB
 compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 messaging.frame.size=4096
 messaging.frame.count=512
 compiler.parallelism=-1
diff --git a/asterixdb/asterix-app/src/main/resources/cc_no_cbo.conf 
b/asterixdb/asterix-app/src/main/resources/cc_no_cbo.conf
index ab028c859b..6b497e28d0 100644
--- a/asterixdb/asterix-app/src/main/resources/cc_no_cbo.conf
+++ b/asterixdb/asterix-app/src/main/resources/cc_no_cbo.conf
@@ -60,6 +60,7 @@ compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
 compiler.sort.parallel=false
+compiler.ordered.fields=false
 compiler.internal.sanitycheck=true
 messaging.frame.size=4096
 messaging.frame.count=512
diff --git 
a/asterixdb/asterix-app/src/main/resources/cc_static_partitioning.conf 
b/asterixdb/asterix-app/src/main/resources/cc_static_partitioning.conf
index 9d28e55725..26db39f861 100644
--- a/asterixdb/asterix-app/src/main/resources/cc_static_partitioning.conf
+++ b/asterixdb/asterix-app/src/main/resources/cc_static_partitioning.conf
@@ -58,6 +58,7 @@ compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
 compiler.sort.parallel=false
+compiler.ordered.fields=false
 compiler.internal.sanitycheck=true
 messaging.frame.size=4096
 messaging.frame.count=512
diff --git 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
index e312b01ff0..54b4004044 100644
--- 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
+++ 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/AsterixHyracksIntegrationUtil.java
@@ -74,6 +74,7 @@ public class AsterixHyracksIntegrationUtil {
     public static final int DEFAULT_HYRACKS_CC_CLUSTER_PORT = 1099;
     public static final String RESOURCES_PATH = 
joinPath(getProjectPath().toString(), "src", "test", "resources");
     public static final String DEFAULT_CONF_FILE = joinPath(RESOURCES_PATH, 
"cc.conf");
+    public static final String DEFAULT_MAIN_CONF_FILE = 
joinPath(RESOURCES_PATH, "cc-main.conf");
     private static final String DEFAULT_STORAGE_PATH = joinPath("target", 
"io", "dir");
     private static String storagePath = DEFAULT_STORAGE_PATH;
     private static final long RESULT_TTL = TimeUnit.MINUTES.toMillis(30);
@@ -469,7 +470,7 @@ public class AsterixHyracksIntegrationUtil {
     private static String getConfPath() {
         String providedPath = System.getProperty("conf.path");
         if (providedPath == null) {
-            return DEFAULT_CONF_FILE;
+            return DEFAULT_MAIN_CONF_FILE;
         }
         return joinPath(RESOURCES_PATH, providedPath);
     }
diff --git 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/CloudStorageIntegrationUtil.java
 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/CloudStorageIntegrationUtil.java
index 47c059918a..6ee625f8d1 100644
--- 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/CloudStorageIntegrationUtil.java
+++ 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/CloudStorageIntegrationUtil.java
@@ -24,7 +24,7 @@ import static org.apache.hyracks.util.file.FileUtil.joinPath;
 public class CloudStorageIntegrationUtil extends AsterixHyracksIntegrationUtil 
{
 
     public static final String RESOURCES_PATH = 
joinPath(getProjectPath().toString(), "src", "test", "resources");
-    public static final String CONFIG_FILE = joinPath(RESOURCES_PATH, 
"cc-cloud-storage.conf");
+    public static final String CONFIG_FILE = joinPath(RESOURCES_PATH, 
"cc-cloud-storage-main.conf");
 
     public static void main(String[] args) throws Exception {
         boolean cleanStart = Boolean.getBoolean("cleanup.start");
diff --git 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
index 8b71d89d03..e9060391ed 100644
--- 
a/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
+++ 
b/asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
@@ -3035,7 +3035,7 @@ public class TestExecutor {
                         LOGGER.error("was expecting the following warnings: ");
                     }
                     for (int i = expectedWarnings.nextSetBit(0); i >= 0; i = 
expectedWarnings.nextSetBit(i + 1)) {
-                        LOGGER.error(expectedWarn.get(i));
+                        LOGGER.error(expectedWarn.get(i).getValue());
                     }
                     throw new Exception(msg);
                 }
diff --git a/asterixdb/asterix-app/src/test/resources/cc-analyze.conf 
b/asterixdb/asterix-app/src/test/resources/cc-analyze.conf
index 93a4d15cc4..c65bca8cf5 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-analyze.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-analyze.conf
@@ -54,4 +54,5 @@ log.dir = logs/
 log.level = INFO
 compiler.groupmemory=64MB
 storage.buffercache.pagesize=32KB
-compiler.internal.sanitycheck=true
\ No newline at end of file
+compiler.internal.sanitycheck=true
+compiler.ordered.fields=false
\ No newline at end of file
diff --git a/asterixdb/asterix-app/src/test/resources/cc-cbotest.conf 
b/asterixdb/asterix-app/src/test/resources/cc-cbotest.conf
index 3a1ac4fcba..aa7e6a27c7 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-cbotest.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-cbotest.conf
@@ -62,6 +62,7 @@ compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
 compiler.sort.parallel=false
+compiler.ordered.fields=false
 compiler.internal.sanitycheck=true
 messaging.frame.size=4096
 messaging.frame.count=512
diff --git 
a/asterixdb/asterix-app/src/test/resources/cc-cloud-storage-azblob.conf 
b/asterixdb/asterix-app/src/test/resources/cc-cloud-storage-azblob.conf
index d11cb5ec5b..4af4fd0485 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-cloud-storage-azblob.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-cloud-storage-azblob.conf
@@ -58,6 +58,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 compiler.internal.sanitycheck=true
 messaging.frame.size=4096
 messaging.frame.count=512
diff --git a/asterixdb/asterix-app/src/test/resources/cc-cloud-storage-gcs.conf 
b/asterixdb/asterix-app/src/test/resources/cc-cloud-storage-gcs.conf
index 518cc71036..325143962a 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-cloud-storage-gcs.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-cloud-storage-gcs.conf
@@ -58,6 +58,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 compiler.internal.sanitycheck=true
 messaging.frame.size=4096
 messaging.frame.count=512
diff --git a/asterixdb/asterix-app/src/test/resources/cc-cloud-storage.conf 
b/asterixdb/asterix-app/src/test/resources/cc-cloud-storage-main.conf
similarity index 98%
copy from asterixdb/asterix-app/src/test/resources/cc-cloud-storage.conf
copy to asterixdb/asterix-app/src/test/resources/cc-cloud-storage-main.conf
index 5672cdecef..a9f2aace9e 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-cloud-storage.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-cloud-storage-main.conf
@@ -71,4 +71,4 @@ cloud.storage.endpoint=http://127.0.0.1:8001
 cloud.storage.anonymous.auth=true
 cloud.storage.cache.policy=selective
 cloud.max.write.requests.per.second=2000
-cloud.max.read.requests.per.second=4000
+cloud.max.read.requests.per.second=4000
\ No newline at end of file
diff --git a/asterixdb/asterix-app/src/test/resources/cc-cloud-storage.conf 
b/asterixdb/asterix-app/src/test/resources/cc-cloud-storage.conf
index 5672cdecef..64fc40fb08 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-cloud-storage.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-cloud-storage.conf
@@ -58,6 +58,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 compiler.internal.sanitycheck=true
 messaging.frame.size=4096
 messaging.frame.count=512
diff --git a/asterixdb/asterix-app/src/test/resources/cc-columnar.conf 
b/asterixdb/asterix-app/src/test/resources/cc-columnar.conf
index 11feeccfba..e01bbbd9a4 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-columnar.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-columnar.conf
@@ -57,6 +57,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 messaging.frame.size=4096
 messaging.frame.count=512
 storage.buffercache.pagesize=32KB
diff --git a/asterixdb/asterix-app/src/test/resources/cc.conf 
b/asterixdb/asterix-app/src/test/resources/cc-main.conf
similarity index 98%
copy from asterixdb/asterix-app/src/test/resources/cc.conf
copy to asterixdb/asterix-app/src/test/resources/cc-main.conf
index 624455c594..cdc3e6bb0f 100644
--- a/asterixdb/asterix-app/src/test/resources/cc.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-main.conf
@@ -60,4 +60,4 @@ compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
 messaging.frame.size=4096
 messaging.frame.count=512
-storage.buffercache.pagesize=32KB
+storage.buffercache.pagesize=32KB
\ No newline at end of file
diff --git a/asterixdb/asterix-app/src/test/resources/cc-single-lookup.conf 
b/asterixdb/asterix-app/src/test/resources/cc-single-lookup.conf
index d59d4d58c5..20549a8de8 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-single-lookup.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-single-lookup.conf
@@ -58,6 +58,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 compiler.batch.lookup=false
 messaging.frame.size=4096
 messaging.frame.count=512
diff --git a/asterixdb/asterix-app/src/test/resources/cc-single-partition.conf 
b/asterixdb/asterix-app/src/test/resources/cc-single-partition.conf
index dab765bfb7..aeb9361c47 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-single-partition.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-single-partition.conf
@@ -49,6 +49,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 messaging.frame.size=4096
 messaging.frame.count=512
 storage.buffercache.pagesize=32KB
diff --git a/asterixdb/asterix-app/src/test/resources/cc-single.conf 
b/asterixdb/asterix-app/src/test/resources/cc-single.conf
index a6f4e8999a..76b29d7e25 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-single.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-single.conf
@@ -45,6 +45,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 messaging.frame.size=4096
 messaging.frame.count=512
 storage.buffercache.pagesize=32KB
diff --git 
a/asterixdb/asterix-app/src/test/resources/cc-small-txn-log-partition.conf 
b/asterixdb/asterix-app/src/test/resources/cc-small-txn-log-partition.conf
index ae91f5e10d..79a805b751 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-small-txn-log-partition.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-small-txn-log-partition.conf
@@ -49,6 +49,7 @@ compiler.sortmemory=320KB
 compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 messaging.frame.size=4096
 messaging.frame.count=512
 txn.log.partitionsize=2MB
diff --git a/asterixdb/asterix-app/src/test/resources/cc-ssl.conf 
b/asterixdb/asterix-app/src/test/resources/cc-ssl.conf
index 547beff9b7..086944b033 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-ssl.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-ssl.conf
@@ -64,6 +64,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 messaging.frame.size=4096
 messaging.frame.count=512
 ssl.enabled=true
diff --git a/asterixdb/asterix-app/src/test/resources/cc-storage.conf 
b/asterixdb/asterix-app/src/test/resources/cc-storage.conf
index 68faa17905..f3a39ab8aa 100644
--- a/asterixdb/asterix-app/src/test/resources/cc-storage.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc-storage.conf
@@ -50,6 +50,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 messaging.frame.size=4096
 messaging.frame.count=512
 storage.buffercache.pagesize=32KB
diff --git a/asterixdb/asterix-app/src/test/resources/cc.conf 
b/asterixdb/asterix-app/src/test/resources/cc.conf
index 624455c594..075d865c79 100644
--- a/asterixdb/asterix-app/src/test/resources/cc.conf
+++ b/asterixdb/asterix-app/src/test/resources/cc.conf
@@ -58,6 +58,7 @@ compiler.groupmemory=160KB
 compiler.joinmemory=256KB
 compiler.textsearchmemory=160KB
 compiler.windowmemory=192KB
+compiler.ordered.fields=false
 messaging.frame.size=4096
 messaging.frame.count=512
 storage.buffercache.pagesize=32KB
diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1/cluster_state_1.1.regexadm
 
b/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1/cluster_state_1.1.regexadm
index 9d538056dc..7c963a7cdc 100644
--- 
a/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1/cluster_state_1.1.regexadm
+++ 
b/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1/cluster_state_1.1.regexadm
@@ -47,7 +47,7 @@
     "compiler\.min\.memory\.allocation" : true,
     "compiler.min.sortmemory" : 524288,
     "compiler.min.windowmemory" : 524288,
-    "compiler.orderfields" : false,
+    "compiler.ordered.fields" : false,
     "compiler\.parallelism" : 0,
     "compiler.queryplanshape" : "zigzag",
     "compiler.runtime.memory.overhead" : 5,
diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_full/cluster_state_1_full.1.regexadm
 
b/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_full/cluster_state_1_full.1.regexadm
index 0b23562c30..50927b4d8f 100644
--- 
a/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_full/cluster_state_1_full.1.regexadm
+++ 
b/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_full/cluster_state_1_full.1.regexadm
@@ -47,7 +47,7 @@
     "compiler\.min\.memory\.allocation" : true,
     "compiler.min.sortmemory" : 524288,
     "compiler.min.windowmemory" : 524288,
-    "compiler.orderfields" : false,
+    "compiler.ordered.fields" : false,
     "compiler\.parallelism" : -1,
     "compiler.queryplanshape" : "zigzag",
     "compiler.runtime.memory.overhead" : 5,
diff --git 
a/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_less/cluster_state_1_less.1.regexadm
 
b/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_less/cluster_state_1_less.1.regexadm
index 0d77d056c5..1b035a0a21 100644
--- 
a/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_less/cluster_state_1_less.1.regexadm
+++ 
b/asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_less/cluster_state_1_less.1.regexadm
@@ -47,7 +47,7 @@
     "compiler\.min\.memory\.allocation" : true,
     "compiler.min.sortmemory" : 524288,
     "compiler.min.windowmemory" : 524288,
-    "compiler.orderfields" : false,
+    "compiler.ordered.fields" : false,
     "compiler\.parallelism" : 3,
     "compiler.queryplanshape" : "zigzag",
     "compiler.runtime.memory.overhead" : 5,
diff --git 
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
 
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
index 6f11e1e9cd..a7e39c9b2a 100644
--- 
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
+++ 
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
@@ -157,7 +157,7 @@ public class CompilerProperties extends AbstractProperties {
                 getRangedIntegerType(0, Integer.MAX_VALUE),
                 128,
                 "Maximum occurrences of a variable allowed in an expression 
for inlining"),
-        COMPILER_ORDERFIELDS(BOOLEAN, AlgebricksConfig.ORDERED_FIELDS, 
"Enable/disable select order list");
+        COMPILER_ORDERED_FIELDS(BOOLEAN, AlgebricksConfig.ORDERED_FIELDS, 
"Enable/disable select order list");
 
         private final IOptionType type;
         private final Object defaultValue;
@@ -242,7 +242,7 @@ public class CompilerProperties extends AbstractProperties {
     public static final String COMPILER_MAX_VARIABLE_OCCURRENCES_INLINING_KEY =
             Option.COMPILER_MAX_VARIABLE_OCCURRENCES_INLINING.ini();
 
-    public static final String COMPILER_ORDERFIELDS_KEY = 
Option.COMPILER_ORDERFIELDS.ini();
+    public static final String COMPILER_ORDERED_FIELDS_KEY = 
Option.COMPILER_ORDERED_FIELDS.ini();
 
     public static final int COMPILER_PARALLELISM_AS_STORAGE = 0;
 
@@ -373,7 +373,7 @@ public class CompilerProperties extends AbstractProperties {
     }
 
     public boolean isOrderedFields() {
-        return accessor.getBoolean(Option.COMPILER_ORDERFIELDS);
+        return accessor.getBoolean(Option.COMPILER_ORDERED_FIELDS);
     }
 
     public int getRuntimeMemoryOverheadPercentage() {
diff --git 
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/OptimizationConfUtil.java
 
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/OptimizationConfUtil.java
index fe5ec9e096..c866d223a6 100644
--- 
a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/OptimizationConfUtil.java
+++ 
b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/OptimizationConfUtil.java
@@ -97,7 +97,7 @@ public class OptimizationConfUtil {
                 compilerProperties.isColumnFilter());
         int maxVariableOccurrencesForInlining =
                 getMaxVariableOccurrencesForInlining(compilerProperties, 
querySpecificConfig, sourceLoc);
-        boolean orderFields = getBoolean(querySpecificConfig, 
CompilerProperties.COMPILER_ORDERFIELDS_KEY,
+        boolean orderFields = getBoolean(querySpecificConfig, 
CompilerProperties.COMPILER_ORDERED_FIELDS_KEY,
                 compilerProperties.isOrderedFields());
 
         PhysicalOptimizationConfig physOptConf = new 
PhysicalOptimizationConfig();
diff --git 
a/asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/OpenRecordConstructorResultType.java
 
b/asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/OpenRecordConstructorResultType.java
index bc3ae60879..8358627c93 100644
--- 
a/asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/OpenRecordConstructorResultType.java
+++ 
b/asterixdb/asterix-om/src/main/java/org/apache/asterix/om/typecomputer/impl/OpenRecordConstructorResultType.java
@@ -68,8 +68,8 @@ public class OpenRecordConstructorResultType implements 
IResultTypeComputer {
         boolean orderFields = AlgebricksConfig.ORDERED_FIELDS;
         if (metadataProvider != null) {
             Map<String, Object> config = metadataProvider.getConfig();
-            orderFields = Boolean.parseBoolean((String) 
config.getOrDefault(CompilerProperties.COMPILER_ORDERFIELDS_KEY,
-                    String.valueOf(orderFields)));
+            orderFields = Boolean.parseBoolean((String) config
+                    
.getOrDefault(CompilerProperties.COMPILER_ORDERED_FIELDS_KEY, 
String.valueOf(orderFields)));
         }
 
         Iterator<Mutable<ILogicalExpression>> argIter = 
f.getArguments().iterator();
diff --git 
a/asterixdb/asterix-server/src/test/resources/NCServiceExecutionIT/cc.conf 
b/asterixdb/asterix-server/src/test/resources/NCServiceExecutionIT/cc.conf
index 1c6d574edb..5b159ea02a 100644
--- a/asterixdb/asterix-server/src/test/resources/NCServiceExecutionIT/cc.conf
+++ b/asterixdb/asterix-server/src/test/resources/NCServiceExecutionIT/cc.conf
@@ -47,4 +47,5 @@ heartbeat.max.misses=25
 
 [common]
 log.dir = ../asterix-server/target/NCServiceExecutionIT
-log.level = INFO
\ No newline at end of file
+log.level = INFO
+compiler.ordered.fields=false
\ No newline at end of file
diff --git a/asterixdb/asterix-server/src/test/resources/NcLifecycleIT/cc.conf 
b/asterixdb/asterix-server/src/test/resources/NcLifecycleIT/cc.conf
index a91622b493..a5a437a477 100644
--- a/asterixdb/asterix-server/src/test/resources/NcLifecycleIT/cc.conf
+++ b/asterixdb/asterix-server/src/test/resources/NcLifecycleIT/cc.conf
@@ -47,3 +47,4 @@ heartbeat.max.misses=25
 [common]
 log.level = INFO
 log.dir=../asterix-server/target/NcLifecycleIT
+compiler.ordered.fields=false
\ No newline at end of file
diff --git a/asterixdb/asterix-server/src/test/resources/ReplicationIT/cc.conf 
b/asterixdb/asterix-server/src/test/resources/ReplicationIT/cc.conf
index b88cbd4595..bef0a3cbf7 100644
--- a/asterixdb/asterix-server/src/test/resources/ReplicationIT/cc.conf
+++ b/asterixdb/asterix-server/src/test/resources/ReplicationIT/cc.conf
@@ -51,3 +51,4 @@ log.dir=../asterix-server/target/ReplicationIT
 replication.enabled=true
 replication.strategy=all
 replication.factor=2
+compiler.ordered.fields=false
\ No newline at end of file
diff --git 
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java
 
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java
index 0b29f7d97b..ccb568cf1f 100644
--- 
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java
+++ 
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java
@@ -46,6 +46,6 @@ public class AlgebricksConfig {
             StorageUtil.getIntSizeInBytes(8, StorageUtil.StorageUnit.KILOBYTE);
     public static final boolean BATCH_LOOKUP_DEFAULT = true;
     public static final boolean COLUMN_FILTER_DEFAULT = true;
-    public static final boolean ORDERED_FIELDS = false;
+    public static final boolean ORDERED_FIELDS = true;
     public static final int MAX_VARIABLE_OCCURRENCES_INLINING_DEFAULT = 128;
 }
diff --git 
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
 
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
index 163fb164b6..7419964080 100644
--- 
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
+++ 
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
@@ -64,7 +64,7 @@ public class PhysicalOptimizationConfig {
     private static final String MIN_WINDOW_FRAMES = "MIN_WINDOW_FRAMES";
     private static final String MAX_VARIABLE_OCCURRENCES_INLINING = 
"MAX_VARIABLE_OCCURRENCES_INLINING";
 
-    private static final String ORDER_FIELDS = "ORDER_FIELDS";
+    private static final String ORDER_FIELDS = "ORDERED_FIELDS";
 
     private final Properties properties = new Properties();
 

Reply via email to