dynamic_migrations 2.1.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6bbdbe66b5d24e600037fcaf794929bca95cb55b6a2bd45de324934ab1ab22b2
-  data.tar.gz: 06dd757a70e427b753e6f5b3f1bdbf970c749947d13ba7771449f595a2968743
+  metadata.gz: b27309525226b33bbcff1b08b3361347e93cff1d182f5620debbe3195b5c1d88
+  data.tar.gz: 670213a55b129e2518e793bc5314fe522f72645da50d5b32355cacfec5cff8fb
 SHA512:
-  metadata.gz: c1f8deb5208188c16ff9b6b0071b9fea82111626336cd73d77e519ce93bb1af4df74090c3e34de49d007d1709f7b7041f95cbfca97dc4676e91e74b19b5ee556
-  data.tar.gz: bfced908cb97de19a678043f6e6fff00faeb11d4dc1ac8b3b15b3e8611f21c121b5ad9a04dae54a8cbf118f9efee9b9462a1b6dcf12904c511d23c15d079ea93
+  metadata.gz: 4227acc9480ac7c9022a0e1f9473627d3b7ab11611d164582b9edf9049928a4a36c1292ce5f43f9f627f9c2c4968e6b8bb4da1ed0dde11b41cd431d6d61022a6
+  data.tar.gz: ef666cf1f797e262469fc080524726d3e70e1f1a52a4dfdb08bc9b9fbc5925671b25cead8013555618ade984bf603e86812ea8689cba2e59a5dd1a8d411664dc

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [2.2.0](https://github.com/craigulliott/dynamic_migrations/compare/v2.1.0...v2.2.0) (2023-07-31)
+
+
+### Features
+
+* using postgres shorthand names such as `numeric(12,2)` for types, and removing the additional column metadata ([86d0113](https://github.com/craigulliott/dynamic_migrations/commit/86d0113aabd4350f278164d18e3c0a611fcf5595))
+
 ## [2.1.0](https://github.com/craigulliott/dynamic_migrations/compare/v2.0.0...v2.1.0) (2023-07-31)
 
 

data/lib/dynamic_migrations/postgres/server/database/differences.rb

@@ -130,16 +130,7 @@ module DynamicMigrations
                 :null,
                 :default,
                 :description,
-                :character_maximum_length,
-                :character_octet_length,
-                :numeric_precision,
-                :numeric_precision_radix,
-                :numeric_scale,
-                :datetime_precision,
-                :interval_type,
-                :udt_schema,
-                :udt_name,
-                :updatable
+                :interval_type
               ]
             end
             # look for any columns in the comparison list which were not in the base list

data/lib/dynamic_migrations/postgres/server/database/loaded_schemas_builder.rb

@@ -22,28 +22,11 @@ module DynamicMigrations
 
                 # add each table column
                 table_definition[:columns].each do |column_name, column_definition|
-                  # we only need these for arrays and user-defined types
-                  # (user-defined is usually ENUMS)
-                  if [:ARRAY, :"USER-DEFINED"].include? column_definition[:data_type]
-                    udt_schema = column_definition[:udt_schema]
-                    udt_name = column_definition[:udt_name]
-                  else
-                    udt_schema = nil
-                    udt_name = nil
-                  end
-
                   table.add_column column_name, column_definition[:data_type],
                     null: column_definition[:null],
                     default: column_definition[:default],
                     description: column_definition[:description],
-                    character_maximum_length: column_definition[:character_maximum_length],
-                    character_octet_length: column_definition[:character_octet_length],
-                    numeric_precision: column_definition[:numeric_precision],
-                    numeric_precision_radix: column_definition[:numeric_precision_radix],
-                    numeric_scale: column_definition[:numeric_scale],
-                    datetime_precision: column_definition[:datetime_precision],
-                    udt_schema: udt_schema,
-                    udt_name: udt_name
+                    interval_type: column_definition[:interval_type]
                 end
 
                 # add any validations

data/lib/dynamic_migrations/postgres/server/database/schema/table/column.rb

@@ -13,23 +13,14 @@ module DynamicMigrations
 
               attr_reader :table
               attr_reader :name
+              attr_reader :data_type
               attr_reader :description
               attr_reader :null
               attr_reader :default
-              attr_reader :data_type
-              attr_reader :character_maximum_length
-              attr_reader :character_octet_length
-              attr_reader :numeric_precision
-              attr_reader :numeric_precision_radix
-              attr_reader :numeric_scale
-              attr_reader :datetime_precision
               attr_reader :interval_type
-              attr_reader :udt_schema
-              attr_reader :udt_name
-              attr_reader :updatable
 
               # initialize a new object to represent a column in a postgres table
-              def initialize source, table, name, data_type, null: true, default: nil, description: nil, character_maximum_length: nil, character_octet_length: nil, numeric_precision: nil, numeric_precision_radix: nil, numeric_scale: nil, datetime_precision: nil, interval_type: nil, udt_schema: nil, udt_name: nil, updatable: true
+              def initialize source, table, name, data_type, null: true, default: nil, description: nil, interval_type: nil
                 super source
                 raise ExpectedTableError, table unless table.is_a? Table
                 @table = table
@@ -38,9 +29,7 @@ module DynamicMigrations
                 @name = name
 
                 @data_type = data_type
-
                 @null = null
-
                 @default = default
 
                 unless description.nil?
@@ -48,38 +37,7 @@ module DynamicMigrations
                   @description = description
                 end
 
-                # apply any defaults for this data type
-                character_maximum_length = character_maximum_length.nil? ? DataTypes.default_for(data_type, :character_maximum_length) : character_maximum_length
-                character_octet_length = character_octet_length.nil? ? DataTypes.default_for(data_type, :character_octet_length) : character_octet_length
-                numeric_precision = numeric_precision.nil? ? DataTypes.default_for(data_type, :numeric_precision) : numeric_precision
-                numeric_precision_radix = numeric_precision_radix.nil? ? DataTypes.default_for(data_type, :numeric_precision_radix) : numeric_precision_radix
-                numeric_scale = numeric_scale.nil? ? DataTypes.default_for(data_type, :numeric_scale) : numeric_scale
-                datetime_precision = datetime_precision.nil? ? DataTypes.default_for(data_type, :datetime_precision) : datetime_precision
-                interval_type = interval_type.nil? ? DataTypes.default_for(data_type, :interval_type) : interval_type
-                udt_schema = udt_schema.nil? ? DataTypes.default_for(data_type, :udt_schema) : udt_schema
-                udt_name = udt_name.nil? ? DataTypes.default_for(data_type, :udt_name) : udt_name
-
-                DataTypes.validate_column_properties!(data_type,
-                  character_maximum_length: character_maximum_length,
-                  character_octet_length: character_octet_length,
-                  numeric_precision: numeric_precision,
-                  numeric_precision_radix: numeric_precision_radix,
-                  numeric_scale: numeric_scale,
-                  datetime_precision: datetime_precision,
-                  interval_type: interval_type,
-                  udt_schema: udt_schema,
-                  udt_name: udt_name)
-
-                @character_maximum_length = character_maximum_length
-                @character_octet_length = character_octet_length
-                @numeric_precision = numeric_precision
-                @numeric_precision_radix = numeric_precision_radix
-                @numeric_scale = numeric_scale
-                @datetime_precision = datetime_precision
                 @interval_type = interval_type
-                @udt_schema = udt_schema
-                @udt_name = udt_name
-                @updatable = updatable
               end
 
               # return true if this column has a description, otherwise false

data/lib/dynamic_migrations/postgres/server/database/structure_loader.rb

@@ -7,88 +7,41 @@ module DynamicMigrations
         module StructureLoader
           def create_database_structure_cache
             connection.exec(<<~SQL)
-              CREATE MATERIALIZED VIEW public.dynamic_migrations_structure_cache as
+              CREATE MATERIALIZED VIEW public.dynamic_migrations_structure_cache AS
                 SELECT
                   -- Name of the schema containing the table
                   schemata.schema_name,
                   -- Name of the table
                   tables.table_name,
                   -- The comment which has been added to the table (if any)
-                  table_description.description as table_description,
+                  table_description.description AS table_description,
                   -- Name of the column
                   columns.column_name,
                   -- The comment which has been added to the column (if any)
-                  column_description.description as column_description,
+                  column_description.description AS column_description,
                   -- Default expression of the column
                   columns.column_default,
                   -- YES if the column is possibly nullable, NO if
                   -- it is known not nullable
                   columns.is_nullable,
-                  -- Data type of the column, if it is a built-in type,
-                  -- or ARRAY if it is some array (in that case, see the
-                  -- view element_types), else USER-DEFINED (in that case,
-                  -- the type is identified in udt_name and associated
-                  -- columns). If the column is based on a domain, this
-                  -- column refers to the type underlying the domain (and
-                  -- the domain is identified in domain_name and associated
-                  -- columns).
-                  columns.data_type,
-                  -- If data_type identifies a character or bit string type,
-                  -- the declared maximum length; null for all other data
-                  -- types or if no maximum length was declared.
-                  columns.character_maximum_length,
-                  -- If data_type identifies a character type, the maximum
-                  -- possible length in octets (bytes) of a datum; null for
-                  -- all other data types. The maximum octet length depends
-                  -- on the declared character maximum length (see above)
-                  -- and the server encoding.
-                  columns.character_octet_length,
-                  -- If data_type identifies a numeric type, this column
-                  -- contains the (declared or implicit) precision of the type
-                  -- for this column. The precision indicates the number of
-                  -- significant digits. It can be expressed in decimal (base 10)
-                  -- or binary (base 2) terms, as specified in the column
-                  -- numeric_precision_radix. For all other data types, this
-                  -- column is null.
-                  columns.numeric_precision,
-                  -- If data_type identifies a numeric type, this column indicates
-                  -- in which base the values in the columns numeric_precision and
-                  -- numeric_scale are expressed. The value is either 2 or 10. For
-                  -- all other data types, this column is null.
-                  columns.numeric_precision_radix,
-                  -- If data_type identifies an exact numeric type, this column
-                  -- contains the (declared or implicit) scale of the type for this
-                  -- column. The scale indicates the number of significant digits to
-                  -- the right of the decimal point. It can be expressed in decimal
-                  -- (base 10) or binary (base 2) terms, as specified in the column
-                  -- numeric_precision_radix. For all other data types, this column
-                  -- is null.
-                  columns.numeric_scale,
-                  -- If data_type identifies a date, time, timestamp, or interval
-                  -- type, this column contains the (declared or implicit) fractional
-                  -- seconds precision of the type for this column, that is, the
-                  -- number of decimal digits maintained following the decimal point
-                  -- in the seconds value. For all other data types, this column is
-                  -- null.
-                  columns.datetime_precision,
+                  -- The formatted data type (such as integer, char(5) or numeric(12,2)[])
+                  CASE
+                  WHEN tables.table_name IS NOT NULL THEN
+                  (
+                    SELECT format_type(atttypid,atttypmod) FROM pg_attribute a
+                      WHERE a.attrelid = concat('"', schemata.schema_name, '"', '.', '"', tables.table_name, '"')::regclass
+                      AND attnum = columns.ordinal_position
+                  )
+                  END AS data_type,
                   -- If data_type identifies an interval type, this column contains
                   -- the specification which fields the intervals include for this
                   -- column, e.g., YEAR TO MONTH, DAY TO SECOND, etc. If no field
                   -- restrictions were specified (that is, the interval accepts all
-                    -- fields), and for all other data types, this field is null.
-                  columns.interval_type,
-                  -- Name of the schema that the column data type (the underlying
-                  --type of the domain, if applicable) is defined in
-                  columns.udt_schema,
-                  -- Name of the column data type (the underlying type of the domain,
-                  -- if applicable)
-                  columns.udt_name,
-                  -- YES if the column is updatable, NO if not (Columns in base tables
-                    -- are always updatable, columns in views not necessarily)
-                  columns.is_updatable
+                  -- fields), and for all other data types, this field is null.
+                  columns.interval_type
                 FROM information_schema.schemata
                 LEFT JOIN information_schema.tables ON schemata.schema_name = tables.table_schema AND left(tables.table_name, 3) != 'pg_'
-                LEFT JOIN information_schema.columns ON tables.table_name = columns.table_name
+                LEFT JOIN information_schema.columns ON tables.table_name = columns.table_name AND schemata.schema_name = columns.table_schema
                 -- required for the column and table description/comment joins
                 LEFT JOIN pg_catalog.pg_statio_all_tables ON pg_statio_all_tables.schemaname = schemata.schema_name AND pg_statio_all_tables.relname = tables.table_name
                 -- required for the table description/comment
@@ -142,16 +95,7 @@ module DynamicMigrations
                   column[:null] = row["is_nullable"] == "YES"
                   column[:default] = row["column_default"]
                   column[:description] = row["column_description"]
-                  column[:character_maximum_length] = row["character_maximum_length"].nil? ? nil : row["character_maximum_length"].to_i
-                  column[:character_octet_length] = row["character_octet_length"].nil? ? nil : row["character_octet_length"].to_i
-                  column[:numeric_precision] = row["numeric_precision"].nil? ? nil : row["numeric_precision"].to_i
-                  column[:numeric_precision_radix] = row["numeric_precision_radix"].nil? ? nil : row["numeric_precision_radix"].to_i
-                  column[:numeric_scale] = row["numeric_scale"].nil? ? nil : row["numeric_scale"].to_i
-                  column[:datetime_precision] = row["datetime_precision"].nil? ? nil : row["datetime_precision"].to_i
                   column[:interval_type] = row["interval_type"].nil? ? nil : row["interval_type"].to_sym
-                  column[:udt_schema] = row["udt_schema"].to_sym
-                  column[:udt_name] = row["udt_name"].to_sym
-                  column[:updatable] = row["is_updatable"] == "YES"
                 end
               end
             end
@@ -168,29 +112,11 @@ module DynamicMigrations
                 table = schema.add_table table_name, table_definition[:description]
 
                 table_definition[:columns].each do |column_name, column_definition|
-                  # we only need these for arrays and user-defined types
-                  # (user-defined is usually ENUMS)
-                  if [:ARRAY, :"USER-DEFINED"].include? column_definition[:data_type]
-                    udt_schema = column_definition[:udt_schema]
-                    udt_name = column_definition[:udt_name]
-                  else
-                    udt_schema = nil
-                    udt_name = nil
-                  end
-
                   table.add_column column_name, column_definition[:data_type],
                     null: column_definition[:null],
                     default: column_definition[:default],
                     description: column_definition[:description],
-                    character_maximum_length: column_definition[:character_maximum_length],
-                    character_octet_length: column_definition[:character_octet_length],
-                    numeric_precision: column_definition[:numeric_precision],
-                    numeric_precision_radix: column_definition[:numeric_precision_radix],
-                    numeric_scale: column_definition[:numeric_scale],
-                    datetime_precision: column_definition[:datetime_precision],
-                    udt_schema: udt_schema,
-                    udt_name: udt_name,
-                    updatable: column_definition[:updatable]
+                    interval_type: column_definition[:interval_type]
                 end
               end
             end
@@ -219,22 +145,6 @@ module DynamicMigrations
             table_names.reject! { |table_name| table_name.start_with? "pg_" }
             table_names.sort
           end
-
-          # returns a list of columns definitions for the provided table
-          def fetch_columns schema_name, table_name
-            rows = connection.exec_params(<<-SQL, [schema_name.to_s, table_name.to_s])
-                  SELECT column_name, is_nullable, data_type, character_octet_length, column_default, numeric_precision, numeric_precision_radix, numeric_scale, udt_schema, udt_name
-                    FROM information_schema.columns
-                  WHERE table_schema = $1
-                    AND table_name = $2;
-            SQL
-            rows.map do |row|
-              {
-                column_name: row["column_name"].to_sym,
-                type: row["data_type"].to_sym
-              }
-            end
-          end
         end
       end
     end

data/lib/dynamic_migrations/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DynamicMigrations
-  VERSION = "2.1.0"
+  VERSION = "2.2.0"
 end

data/lib/dynamic_migrations.rb

@@ -36,7 +36,6 @@ require "dynamic_migrations/postgres/server/database/schema/table/unique_constra
 
 require "dynamic_migrations/postgres/server"
 require "dynamic_migrations/postgres/connections"
-require "dynamic_migrations/postgres/data_types"
 
 module DynamicMigrations
   class Error < StandardError

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dynamic_migrations
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.2.0
 platform: ruby
 authors:
 - Craig Ulliott
@@ -73,7 +73,6 @@ files:
 - lib/dynamic_migrations/module_included_into_unexpected_target_error.rb
 - lib/dynamic_migrations/postgres.rb
 - lib/dynamic_migrations/postgres/connections.rb
-- lib/dynamic_migrations/postgres/data_types.rb
 - lib/dynamic_migrations/postgres/server.rb
 - lib/dynamic_migrations/postgres/server/database.rb
 - lib/dynamic_migrations/postgres/server/database/configured_schemas.rb

data/lib/dynamic_migrations/postgres/data_types.rb

@@ -1,320 +0,0 @@
-# frozen_string_literal: true
-
-module DynamicMigrations
-  module Postgres
-    module DataTypes
-      class MissingRequiredAttributeError < StandardError
-        def initialize data_type, attribute
-          super "Missing required attribute `#{attribute}` for data_type `#{data_type}`"
-        end
-      end
-
-      class UnexpectedPropertyNameError < StandardError
-        def initialize attribute
-          super "Unexpected property `#{attribute}`"
-        end
-      end
-
-      class UnexpectedPropertyError < StandardError
-        def initialize data_type, attribute, value
-          super "Unexpected property `#{attribute}` with value `#{value}` for data_type `#{data_type}`"
-        end
-      end
-
-      class UnsupportedTypeError < StandardError
-        def initialize data_type
-          super "Unsupported type `#{data_type}`"
-        end
-      end
-
-      ATTRIBUTE_NAMES = [
-        :character_maximum_length,
-        :character_octet_length,
-        :numeric_precision,
-        :numeric_precision_radix,
-        :numeric_scale,
-        :datetime_precision,
-        :interval_type,
-        :udt_schema,
-        :udt_name
-      ].freeze
-
-      DATA_TYPES = {
-        ARRAY: {
-          description: "binary data (“byte array”)",
-          required: [
-            :udt_schema,
-            :udt_name
-          ]
-        },
-        "USER-DEFINED": {
-          description: "binary data (“byte array”)",
-          required: [
-            :udt_schema,
-            :udt_name
-          ]
-        },
-        bigint: {
-          description: "signed eight-byte integer",
-          required: [
-            :numeric_precision,
-            :numeric_precision_radix,
-            :numeric_scale
-          ],
-          defaults: {
-            numeric_precision: 64,
-            numeric_precision_radix: 2,
-            numeric_scale: 0
-          }
-        },
-        # skipping this, in my tests it automatically turned into a bigint
-        # bigserial: {
-        #   description: "autoincrementing eight-byte integer"
-        # },
-        bit: {
-          description: "fixed-length bit string",
-          args: "[ (n) ]",
-          required: [
-            :character_maximum_length
-          ]
-        },
-        "bit varying": {
-          description: "variable-length bit string",
-          args: "[ (n) ]"
-        },
-        boolean: {
-          description: "logical Boolean (true/false)",
-          required: []
-        },
-        box: {
-          description: "rectangular box on a plane"
-        },
-        bytea: {
-          description: "binary data (“byte array”)"
-        },
-        character: {
-          description: "fixed-length character string",
-          args: "[ (n) ]",
-          required: [
-            :character_maximum_length,
-            :character_octet_length
-          ]
-        },
-        "character varying": {
-          description: "variable-length character string",
-          args: "[ (n) ]",
-          required: [
-            :character_octet_length
-          ],
-          defaults: {
-            character_octet_length: 1073741824
-          }
-        },
-        cidr: {
-          description: "IPv4 or IPv6 network address"
-        },
-        circle: {
-          description: "circle on a plane"
-        },
-        date: {
-          description: "calendar date (year, month, day)",
-          required: [
-            :datetime_precision
-          ]
-        },
-        "double precision": {
-          description: "double precision floating-point number (8 bytes)",
-          required: [
-            :numeric_precision,
-            :numeric_precision_radix
-          ]
-        },
-        inet: {
-          description: "IPv4 or IPv6 host address"
-        },
-        integer: {
-          description: "signed four-byte integer",
-          required: [
-            :numeric_precision,
-            :numeric_precision_radix,
-            :numeric_scale
-          ],
-          defaults: {
-            numeric_precision: 32,
-            numeric_precision_radix: 2,
-            numeric_scale: 0
-          }
-        },
-        interval: {
-          description: "time span",
-          args: "[ fields ] [ (p) ]",
-          required: [
-            :datetime_precision
-          ]
-        },
-        json: {
-          description: "textual JSON data"
-        },
-        jsonb: {
-          description: "binary JSON data, decomposed"
-        },
-        line: {
-          description: "infinite line on a plane"
-        },
-        lseg: {
-          description: "line segment on a plane"
-        },
-        macaddr: {
-          description: "MAC (Media Access Control) address"
-        },
-        macaddr8: {
-          description: "MAC (Media Access Control) address (EUI-64 format)"
-        },
-        money: {
-          description: "currency amount"
-        },
-        numeric: {
-          description: "exact numeric of selectable precision",
-          args: "[ (p, s) ]",
-          required: [
-            :numeric_precision_radix
-          ],
-          optional: [
-            :numeric_precision,
-            :numeric_scale
-          ]
-        },
-        path: {
-          description: "geometric path on a plane"
-        },
-        pg_lsn: {
-          description: "PostgreSQL Log Sequence Number"
-        },
-        pg_snapshot: {
-          description: "user-level transaction ID snapshot"
-        },
-        point: {
-          description: "geometric point on a plane"
-        },
-        polygon: {
-          description: "closed geometric path on a plane"
-        },
-        real: {
-          description: "single precision floating-point number (4 bytes)",
-          required: [
-            :numeric_precision,
-            :numeric_precision_radix
-          ]
-        },
-        smallint: {
-          description: "signed two-byte",
-          required: [
-            :numeric_precision,
-            :numeric_precision_radix,
-            :numeric_scale
-          ]
-        },
-        smallserial: {
-          description: "autoincrementing two-byte"
-        },
-        serial: {
-          description: "autoincrementing four-byte"
-        },
-        text: {
-          description: "variable-length character string",
-          required: [
-            :character_octet_length
-          ],
-          defaults: {
-            character_octet_length: 1073741824
-          }
-        },
-        "time without time zone": {
-          description: "time of day (no time zone)",
-          args: "[ (p) ]",
-          required: [
-            :datetime_precision
-          ]
-        },
-        "time with time zone": {
-          description: "time of day, including time zone",
-          args: "[ (p) ]",
-          required: [
-            :datetime_precision
-          ]
-        },
-        "timestamp without time zone": {
-          description: "date and time (no time zone)",
-          args: "[ (p) ]",
-          required: [
-            :datetime_precision
-          ]
-        },
-        "timestamp with time zone": {
-          description: "date and time, including time zone",
-          args: "[ (p) ]",
-          required: [
-            :datetime_precision
-          ]
-        },
-        tsquery: {
-          description: "text search query"
-        },
-        tsvector: {
-          description: "text search document"
-        },
-        txid_snapshot: {
-          description: "user-level transaction ID snapshot (deprecated; see pg_snapshot)"
-        },
-        uuid: {
-          description: "universally unique identifier"
-        },
-        xml: {
-          description: "XML data"
-        }
-      }.freeze
-
-      def self.default_for data_type, attribute
-        raise ExpectedSymbolError, data_type unless data_type.is_a? Symbol
-        raise ExpectedSymbolError, attribute unless attribute.is_a? Symbol
-        raise UnsupportedTypeError, data_type unless DATA_TYPES.key? data_type
-
-        unless ATTRIBUTE_NAMES.include? attribute
-          raise UnexpectedPropertyNameError.new attribute
-        end
-
-        defaults = DATA_TYPES[data_type][:defaults]
-        defaults && defaults[attribute] || nil
-      end
-
-      def self.validate_type_exists! data_type
-        raise ExpectedSymbolError, data_type unless data_type.is_a? Symbol
-        raise UnsupportedTypeError, data_type unless DATA_TYPES.key? data_type
-        true
-      end
-
-      def self.validate_column_properties! data_type, **column_options
-        validate_type_exists! data_type
-
-        required_attributes = DATA_TYPES[data_type][:required] || []
-        optional_attributes = DATA_TYPES[data_type][:optional] || []
-        possible_attributes = required_attributes + optional_attributes
-
-        # assert all required attributes are present
-        required_attributes.each do |attribute|
-          unless column_options.key?(attribute) && !column_options[attribute].nil?
-            raise MissingRequiredAttributeError.new data_type, attribute
-          end
-        end
-
-        # assert no unexpected attributes are present
-        column_options.each do |key, value|
-          unless value.nil? || possible_attributes.include?(key)
-            raise UnexpectedPropertyError.new data_type, key, value
-          end
-        end
-        true
-      end
-    end
-  end
-end