MDEV-40388: sequence.simple fails on replay

sequence table was determined using tbl->table->s->sequence. The
intention was to not record sequence table information. However, the
check was only partially correct.

Instead, we could compare using the engine name like
if (!tbl->is_view() && tbl->table)
{
  handlerton *hton= tbl->table->file->partition_ht();
  const LEX_CSTRING *engine_name= hton_name(hton);
  if (strcmp(engine_name->str, "SEQUENCE") == 0 ||
      strcmp(engine_name->str, "SQL_SEQUENCE") == 0)
    continue;
}
This makes the sequence.simple succeed

Author: bsrikanth-mariadb

mysql-test/main/opt_context_store_stats.result

@@ -591,3 +591,18 @@ table_name	file_stat_records	index_name	rec_per_key
 index_name	ranges	num_rows	max_index_blocks	max_row_blocks
 # == End of optimizer context
 drop table s1;
+#
+# MDEV-40388: sequence.simple fails on replay
+# Table context should *not* be recorded for seq 
+#
+set optimizer_record_context=ON;
+explain select * from seq_15_to_1_step_12345;
+id	select_type	table	type	possible_keys	key	key_len	ref	rows	Extra
+1	SIMPLE	seq_15_to_1_step_12345	ALL	NULL	NULL	NULL	NULL	1	
+# == Optimizer Context
+# === Tables
+# Tables in the context
+table_name	file_stat_records	index_name	rec_per_key
+# === Range accesses
+index_name	ranges	num_rows	max_index_blocks	max_row_blocks
+# == End of optimizer context

mysql-test/main/opt_context_store_stats.test

@@ -432,3 +432,12 @@ EXPLAIN select * from s1;
 
 --source include/opt_context_list_tables_and_ranges.inc
 drop table s1;
+
+--echo #
+--echo # MDEV-40388: sequence.simple fails on replay
+--echo # Table context should *not* be recorded for seq 
+--echo #
+set optimizer_record_context=ON;
+explain select * from seq_15_to_1_step_12345;
+
+--source include/opt_context_list_tables_and_ranges.inc

sql/opt_context_store_replay.cc

@@ -746,8 +746,16 @@ bool Optimizer_context_recorder::dump_sql_script(THD* thd, String &sql_script)
     /*
       Sequence table doesn't need CREATE TABLE or contain any stats
     */
-    if (tbl->table && tbl->table->s && tbl->table->s->sequence)
-      continue;
+    if (!tbl->is_view() && tbl->table)
+    {
+      handlerton *hton= tbl->table->file->partition_ht();
+      const LEX_CSTRING *engine_name= hton_name(hton);
+      if (strcmp(engine_name->str, "SEQUENCE") == 0 ||
+          strcmp(engine_name->str, "SQL_SEQUENCE") == 0)
+      {
+        continue;
+      }
+    }
 
     /*
       A query can use the same table multiple times. Do not dump the