Update kernel/src/schema/diff.rs

murali-db · OussamaSaoudi · web-flow · commit 4802cf68caa7 · 2025-10-16T05:43:14.000+05:30
Co-authored-by: OussamaSaoudi &lt;45303303+OussamaSaoudi@users.noreply.github.com&gt;
diff --git a/kernel/src/schema/diff.rs b/kernel/src/schema/diff.rs
@@ -522,10 +522,8 @@ fn compute_field_update(
     }
 
     // Check for nullability change - distinguish between tightening and loosening
-    match (before.field.nullable, after.field.nullable) {
-        (true, false) => changes.push(FieldChangeType::NullabilityTightened), // Breaking
-        (false, true) => changes.push(FieldChangeType::NullabilityLoosened),  // Safe
-        _ => {}                                                               // No change
+    if let Some(change) = check_container_nullability_change(before.field.nullable, after.field.nullable) {
+        changes.push(change);
     }
 
     // Validate physical name consistency

Original file line number	Diff line number	Diff line change
`@@ -522,10 +522,8 @@ fn compute_field_update(`
`522`	`522`	`}`
`523`	`523`
`524`	`524`	`// Check for nullability change - distinguish between tightening and loosening`
`525`		`- match (before.field.nullable, after.field.nullable) {`
`526`		`- (true, false) => changes.push(FieldChangeType::NullabilityTightened), // Breaking`
`527`		`- (false, true) => changes.push(FieldChangeType::NullabilityLoosened), // Safe`
`528`		`- _ => {} // No change`
	`525`	`+ if let Some(change) = check_container_nullability_change(before.field.nullable, after.field.nullable) {`
	`526`	`+ changes.push(change);`
`529`	`527`	`}`
`530`	`528`
`531`	`529`	`// Validate physical name consistency`