Skip to content

Commit 487f386

Browse files
committed
Bug#25881783: FAILURE TO INSTALL TABLE WITH FK FROM SDI
Add SDI serialization/deserialization of "referenced table catalog name" for foreign keys and correct typo for "referenced table schema name". This is required for proper SDI usage for foreign keys. No testcase added since MyISAM does not support FKs and InnoDB does not yet use SDIs.
1 parent 830d64a commit 487f386

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

sql/dd/impl/types/foreign_key_impl.cc

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -278,6 +278,9 @@ Foreign_key_impl::serialize(Sdi_wcontext *wctx, Sdi_writer *w) const
278278

279279
write_opx_reference(w, m_unique_constraint, STRING_WITH_LEN("unique_constraint_opx"));
280280

281+
write(w, m_referenced_table_catalog_name,
282+
STRING_WITH_LEN("referenced_table_catalog_name"));
283+
281284
write(w, m_referenced_table_schema_name,
282285
STRING_WITH_LEN("referenced_table_schema_name"));
283286

@@ -299,7 +302,8 @@ Foreign_key_impl::deserialize(Sdi_rcontext *rctx, const RJ_Value &val)
299302

300303
read_opx_reference(rctx, &m_unique_constraint, val, "unique_constraint_opx");
301304

302-
read(&m_referenced_table_schema_name, val, "referenced_table_shema_name");
305+
read(&m_referenced_table_catalog_name, val, "referenced_table_catalog_name");
306+
read(&m_referenced_table_schema_name, val, "referenced_table_schema_name");
303307
read(&m_referenced_table_name, val, "referenced_table_name");
304308
deserialize_each(rctx, [this] () { return add_element(); },
305309
val, "elements");

0 commit comments

Comments
 (0)