make Selectable a sealed class

Author: gavinking

hibernate-core/src/main/java/org/hibernate/id/ExportableColumn.java

@@ -0,0 +1,149 @@
+/*
+ * SPDX-License-Identifier: LGPL-2.1-or-later
+ * Copyright Red Hat Inc. and Hibernate Authors
+ */
+package org.hibernate.id.enhanced;
+
+import org.hibernate.FetchMode;
+import org.hibernate.MappingException;
+import org.hibernate.boot.model.relational.Database;
+import org.hibernate.boot.spi.MetadataBuildingContext;
+import org.hibernate.internal.util.collections.ArrayHelper;
+import org.hibernate.mapping.Column;
+import org.hibernate.mapping.Selectable;
+import org.hibernate.mapping.Table;
+import org.hibernate.mapping.Value;
+import org.hibernate.mapping.ValueVisitor;
+import org.hibernate.service.ServiceRegistry;
+import org.hibernate.type.BasicType;
+import org.hibernate.type.MappingContext;
+import org.hibernate.type.Type;
+
+import java.util.List;
+
+class ExportableColumnHelper {
+
+	static Column column(Database database, Table table, String segmentColumnName, BasicType<?> type, String typeName) {
+		final Column column = new Column( segmentColumnName );
+		column.setSqlType( typeName );
+		column.setValue( new Value() {
+			@Override
+			public Value copy() {
+				return this;
+			}
+
+			@Override
+			public int getColumnSpan() {
+				return 1;
+			}
+
+			@Override
+			public List<Selectable> getSelectables() {
+				return List.of( column );
+			}
+
+			@Override
+			public List<Column> getColumns() {
+				return List.of( column );
+			}
+
+			@Override
+			public Type getType() {
+				return type;
+			}
+
+			@Override
+			public FetchMode getFetchMode() {
+				return null;
+			}
+
+			@Override
+			public Table getTable() {
+				return table;
+			}
+
+			@Override
+			public boolean hasFormula() {
+				return false;
+			}
+
+			@Override
+			public boolean isAlternateUniqueKey() {
+				return false;
+			}
+
+			@Override
+			public boolean isNullable() {
+				return false;
+			}
+
+			@Override
+			public boolean[] getColumnInsertability() {
+				return ArrayHelper.TRUE;
+			}
+
+			@Override
+			public boolean hasAnyInsertableColumns() {
+				return true;
+			}
+
+			@Override
+			public boolean[] getColumnUpdateability() {
+				return ArrayHelper.TRUE;
+			}
+
+			@Override
+			public boolean hasAnyUpdatableColumns() {
+				return true;
+			}
+
+			@Override
+			public void createForeignKey() {
+			}
+
+			@Override
+			public void createUniqueKey(MetadataBuildingContext context) {
+			}
+
+			@Override
+			public boolean isSimpleValue() {
+				return true;
+			}
+
+			@Override
+			public boolean isValid(MappingContext mappingContext) throws MappingException {
+				return false;
+			}
+
+			@Override
+			public void setTypeUsingReflection(String className, String propertyName) {
+			}
+
+			@Override
+			public Object accept(ValueVisitor visitor) {
+				return null;
+			}
+
+			@Override
+			public boolean isSame(Value value) {
+				return this == value;
+			}
+
+			@Override
+			public ServiceRegistry getServiceRegistry() {
+				return database.getServiceRegistry();
+			}
+
+			@Override
+			public boolean isColumnInsertable(int index) {
+				return true;
+			}
+
+			@Override
+			public boolean isColumnUpdateable(int index) {
+				return true;
+			}
+		} );
+		return column;
+	}
+}

hibernate-core/src/main/java/org/hibernate/id/enhanced/ExportableColumnHelper.java

@@ -37,7 +37,6 @@
 import org.hibernate.event.monitor.spi.EventMonitor;
 import org.hibernate.event.monitor.spi.DiagnosticEvent;
 import org.hibernate.generator.GeneratorCreationContext;
-import org.hibernate.id.ExportableColumn;
 import org.hibernate.id.IdentifierGeneratorHelper;
 import org.hibernate.id.IntegralDataTypeHolder;
 import org.hibernate.id.PersistentIdentifierGenerator;
@@ -48,10 +47,12 @@
 import org.hibernate.mapping.Table;
 import org.hibernate.service.ServiceRegistry;
 import org.hibernate.stat.spi.StatisticsImplementor;
+import org.hibernate.type.BasicType;
 import org.hibernate.type.BasicTypeRegistry;
 import org.hibernate.type.StandardBasicTypes;
 import org.hibernate.type.Type;
 
+import org.hibernate.type.descriptor.sql.spi.DdlTypeRegistry;
 import org.jboss.logging.Logger;
 
 import static java.util.Collections.singletonMap;
@@ -695,14 +696,13 @@ public void registerExportables(Database database) {
 
 			final BasicTypeRegistry basicTypeRegistry = database.getTypeConfiguration().getBasicTypeRegistry();
 			// todo : not sure the best solution here.  do we add the columns if missing?  other?
-			final Column segmentColumn = new ExportableColumn(
+			final DdlTypeRegistry ddlTypeRegistry = database.getTypeConfiguration().getDdlTypeRegistry();
+			final Column segmentColumn = ExportableColumnHelper.column(
 					database,
 					table,
 					segmentColumnName,
 					basicTypeRegistry.resolve( StandardBasicTypes.STRING ),
-					database.getTypeConfiguration()
-							.getDdlTypeRegistry()
-							.getTypeName( Types.VARCHAR, Size.length( segmentValueLength ) )
+					ddlTypeRegistry.getTypeName( Types.VARCHAR, Size.length( segmentValueLength ) )
 			);
 			segmentColumn.setNullable( false );
 			table.addColumn( segmentColumn );
@@ -711,11 +711,13 @@ public void registerExportables(Database database) {
 			table.setPrimaryKey( new PrimaryKey( table ) );
 			table.getPrimaryKey().addColumn( segmentColumn );
 
-			final Column valueColumn = new ExportableColumn(
+			final BasicType<?> type = basicTypeRegistry.resolve( StandardBasicTypes.LONG );
+			final Column valueColumn = ExportableColumnHelper.column(
 					database,
 					table,
 					valueColumnName,
-					basicTypeRegistry.resolve( StandardBasicTypes.LONG )
+					type,
+					ddlTypeRegistry.getTypeName( type.getJdbcType().getDdlTypeCode(), database.getDialect() )
 			);
 			table.addColumn( valueColumn );
 		}