datetimetz_count.patch

Alexandru Lungu, 05/16/2023 02:23 AM

     **     ECF 20221207 Changed heuristic to extract schema name from full DMO interface name.
     ** 003 AL2 20230120 Compute if the underlying DMO requires extra computed columns.
     ** 004 AL2 20230317 Extents and date-time extra column are not computed columns; it is safe to exclude them.
                         Avoid lazy initializing hasComputedColumns to avoid concurrency issues.
     **                  Avoid lazy initializing hasComputedColumns to avoid concurrency issues.
     ** 005 AL2 20230516 Cache the number of datetime-tz fields.
     */
     /*
-...
        /** The set of properties and their metadata, indexed by their in-memory positions. */
        private Pair<Property, PropertyMeta>[] properties = null;
        /** Number of fields of type datetime-tz. This is used to retrieve the number of tz offset columns */
        private Integer datetimeTzCount = null;
        /** The unique id of this DMO. */
        private final int id = idSeed.incrementAndGet();
-...
           return Category.VST.name().equals(category) && systemTable.fileName.equalsIgnoreCase(legacyTable);
+       }
        /**
         * Retrieve the number of fields of type datetime-tz. This should be used to detect how many
         * additional tz offset columns are generated.
+        *
         * @return   The number of datetime-tz fields.
         */
        public int getDatetimeTzCount()
+       {
           if (datetimeTzCount == null)
+          {
              int counter = 0;
              Set<Map.Entry<String, Property>> fieldEntries = propsByName.entrySet();
              for (Map.Entry<String, Property> field : fieldEntries)
+             {
                 if (field.getValue()._isDatetimeTz)
+                {
                    counter++; // datetimetz use two columns per property
+                }
+             }
              datetimeTzCount = counter;
+          }
           return datetimeTzCount;
+       }
        /**
         * Check if this DMO has computed columns in the underlying database representation. This is

     **                  Fixed second pass query generation for queries with CONTAINS operator.
     ** 006 DDF 20230306 Skip analyzing propertyStr too see whether it is a ComputedColumn when the
     **                  dialect doesn't require computed columns (refs: #7108).
     ** 006 OM  20230428 Made sure [aliases] is balanced. Avoid double-emit or skipping of FQLAst tokens.
     *                   Fixed handling of nested SUBSELECTs. Local optimizations. Bit of code cleanup.
     ** 007 OM  20230428 Made sure [aliases] is balanced. Avoid double-emit or skipping of FQLAst tokens.
     **                  Fixed handling of nested SUBSELECTs. Local optimizations. Bit of code cleanup.
     ** 008 AL2 20230516 Include the number of datetime-tz offset fields in the field counter.
     */
     /*
-...
                 sb.append(", ");
+             }
              sb.append(sqlTableAliasName).append(".*");
              fieldCounter = dmoMeta.propsByName.size();
              fieldCounter = dmoMeta.propsByName.size() + dmoMeta.getDatetimeTzCount();
              if (rowStructure != null)
+             {

Project

General

Profile

FWD » Core Development » Database

datetimetz_count.patch