HHH-19542 Ensure columns in embeddables default to correct table

Author: beikov

hibernate-core/src/main/java/org/hibernate/boot/model/internal/AnnotatedColumns.java

@@ -111,7 +111,7 @@ public boolean isSecondary() {
 		final String explicitTableName = firstColumn.getExplicitTableName();
 		//note: checkPropertyConsistency() is responsible for ensuring they all have the same table name
 		return isNotEmpty( explicitTableName )
-			&& !getPropertyHolder().getTable().getName().equals( explicitTableName );
+			&& !getOwnerTable().getName().equals( explicitTableName );
 	}
 
 	/**
@@ -130,10 +130,18 @@ public Table getTable() {
 			// all the columns have to be mapped to the same table
 			// even though at the annotation level it looks like
 			// they could each specify a different table
-			return isSecondary() ? getJoin().getTable() : getPropertyHolder().getTable();
+			return isSecondary() ? getJoin().getTable() : getOwnerTable();
 		}
 	}
 
+	private Table getOwnerTable() {
+		PropertyHolder holder = getPropertyHolder();
+		while ( holder instanceof ComponentPropertyHolder componentPropertyHolder ) {
+			holder = componentPropertyHolder.parent;
+		}
+		return holder.getTable();
+	}
+
 	public void setTable(Table table) {
 		this.table = table;
 	}

hibernate-core/src/main/java/org/hibernate/boot/model/internal/ComponentPropertyHolder.java

@@ -4,16 +4,12 @@
  */
 package org.hibernate.boot.model.internal;
 
-import java.util.ArrayList;
 import java.util.HashMap;
-import java.util.List;
 import java.util.Map;
-import java.util.Objects;
 
 import org.hibernate.AnnotationException;
 import org.hibernate.boot.spi.MetadataBuildingContext;
 import org.hibernate.boot.spi.PropertyData;
-import org.hibernate.internal.util.StringHelper;
 import org.hibernate.mapping.AggregateColumn;
 import org.hibernate.mapping.Component;
 import org.hibernate.mapping.Join;
@@ -72,7 +68,6 @@ public class ComponentPropertyHolder extends AbstractPropertyHolder {
 
 	private final String embeddedAttributeName;
 	private final Map<String,AttributeConversionInfo> attributeConversionInfoMap;
-	private final List<AnnotatedColumn> annotatedColumns;
 
 	public ComponentPropertyHolder(
 			Component component,
@@ -99,12 +94,6 @@ public ComponentPropertyHolder(
 			this.embeddedAttributeName = "";
 			this.attributeConversionInfoMap = processAttributeConversions( inferredData.getClassOrElementType() );
 		}
-
-		if ( parent instanceof ComponentPropertyHolder parentHolder ) {
-			this.annotatedColumns = parentHolder.annotatedColumns;
-		} else {
-			this.annotatedColumns = new ArrayList<>();
-		}
 	}
 
 	/**
@@ -236,7 +225,12 @@ public void addProperty(Property property, MemberDetails attributeMemberDetails,
 		// Check table matches between the component and the columns
 		// if not, change the component table if no properties are set
 		// if a property is set already the core cannot support that
-		final Table table = property.getValue().getTable();
+		assert property.getValue().getTable() == columns.getTable();
+		setTable( property.getValue().getTable() );
+		addProperty( property, attributeMemberDetails, declaringClass );
+	}
+
+	void setTable(Table table) {
 		if ( !table.equals( getTable() ) ) {
 			if ( component.getPropertySpan() == 0 ) {
 				component.setTable( table );
@@ -248,27 +242,8 @@ public void addProperty(Property property, MemberDetails attributeMemberDetails,
 						+ " (all properties of the embeddable class must map to the same table)"
 				);
 			}
-		}
-		if ( columns != null ) {
-			annotatedColumns.addAll( columns.getColumns() );
-		}
-		addProperty( property, attributeMemberDetails, declaringClass );
-	}
-
-	public void checkPropertyConsistency() {
-		if ( annotatedColumns.size() > 1 ) {
-			for ( int currentIndex = 1; currentIndex < annotatedColumns.size(); currentIndex++ ) {
-				final AnnotatedColumn current = annotatedColumns.get( currentIndex );
-				final AnnotatedColumn previous = annotatedColumns.get( currentIndex - 1 );
-				if ( !Objects.equals(
-						StringHelper.nullIfEmpty( current.getExplicitTableName() ),
-						StringHelper.nullIfEmpty( previous.getExplicitTableName() ) ) ) {
-					throw new AnnotationException(
-							"Embeddable class '" + component.getComponentClassName()
-							+ "' has properties mapped to two different tables"
-							+ " (all properties of the embeddable class must map to the same table)"
-					);
-				}
+			if ( parent instanceof ComponentPropertyHolder parentComponentHolder ) {
+				parentComponentHolder.setTable( table );
 			}
 		}
 	}

hibernate-core/src/main/java/org/hibernate/boot/model/internal/EmbeddableBinder.java

@@ -455,7 +455,7 @@ static Component fillEmbeddable(
 		if ( LOG.isDebugEnabled() ) {
 			LOG.debug( "Binding component with path: " + subpath );
 		}
-		final ComponentPropertyHolder subholder = buildPropertyHolder(
+		final PropertyHolder subholder = buildPropertyHolder(
 				component,
 				subpath,
 				inferredData,
@@ -579,8 +579,6 @@ else if ( member.hasDirectAnnotationUsage( GeneratedValue.class ) ) {
 			}
 		}
 
-		subholder.checkPropertyConsistency();
-
 		if ( compositeUserType != null ) {
 			processCompositeUserType( component, compositeUserType );
 		}

hibernate-core/src/main/java/org/hibernate/boot/model/internal/PropertyHolderBuilder.java

@@ -47,7 +47,7 @@ public static PropertyHolder buildPropertyHolder(
 	 *
 	 * @return PropertyHolder
 	 */
-	public static ComponentPropertyHolder buildPropertyHolder(
+	public static PropertyHolder buildPropertyHolder(
 			Component component,
 			String path,
 			PropertyData inferredData,