Commit ef0b91ea authored by Andrei Elkin's avatar Andrei Elkin

MDEV-17803: ulonglongization of table_mapping entry::table_id to fix windows...

MDEV-17803: ulonglongization of table_mapping entry::table_id to fix windows compilation in particular.
parent 5d48ea7d
...@@ -11332,8 +11332,8 @@ int Table_map_log_event::do_apply_event(rpl_group_info *rgi) ...@@ -11332,8 +11332,8 @@ int Table_map_log_event::do_apply_event(rpl_group_info *rgi)
table_list->updating= 1; table_list->updating= 1;
table_list->required_type= FRMTYPE_TABLE; table_list->required_type= FRMTYPE_TABLE;
DBUG_PRINT("debug", ("table: %s is mapped to %u", table_list->table_name, DBUG_PRINT("debug", ("table: %s is mapped to %llu", table_list->table_name,
table_list->table_id)); table_list->table_id));
table_list->master_had_triggers= ((m_flags & TM_BIT_HAS_TRIGGERS_F) ? 1 : 0); table_list->master_had_triggers= ((m_flags & TM_BIT_HAS_TRIGGERS_F) ? 1 : 0);
DBUG_PRINT("debug", ("table->master_had_triggers=%d", DBUG_PRINT("debug", ("table->master_had_triggers=%d",
(int)table_list->master_had_triggers)); (int)table_list->master_had_triggers));
......
...@@ -43,7 +43,7 @@ table_mapping::table_mapping() ...@@ -43,7 +43,7 @@ table_mapping::table_mapping()
constructor is called at startup only. constructor is called at startup only.
*/ */
(void) my_hash_init(&m_table_ids,&my_charset_bin,TABLE_ID_HASH_SIZE, (void) my_hash_init(&m_table_ids,&my_charset_bin,TABLE_ID_HASH_SIZE,
offsetof(entry,table_id),sizeof(ulong), offsetof(entry,table_id),sizeof(ulonglong),
0,0,0); 0,0,0);
/* We don't preallocate any block, this is consistent with m_free=0 above */ /* We don't preallocate any block, this is consistent with m_free=0 above */
init_alloc_root(&m_mem_root, TABLE_ID_HASH_SIZE*sizeof(entry), 0, MYF(0)); init_alloc_root(&m_mem_root, TABLE_ID_HASH_SIZE*sizeof(entry), 0, MYF(0));
...@@ -59,20 +59,20 @@ table_mapping::~table_mapping() ...@@ -59,20 +59,20 @@ table_mapping::~table_mapping()
free_root(&m_mem_root, MYF(0)); free_root(&m_mem_root, MYF(0));
} }
TABLE* table_mapping::get_table(ulong table_id) TABLE* table_mapping::get_table(ulonglong table_id)
{ {
DBUG_ENTER("table_mapping::get_table(ulong)"); DBUG_ENTER("table_mapping::get_table(ulong)");
DBUG_PRINT("enter", ("table_id: %lu", table_id)); DBUG_PRINT("enter", ("table_id: %llu", table_id));
entry *e= find_entry(table_id); entry *e= find_entry(table_id);
if (e) if (e)
{ {
DBUG_PRINT("info", ("tid %lu -> table 0x%lx (%s)", DBUG_PRINT("info", ("tid %llu -> table 0x%lx (%s)",
table_id, (long) e->table, table_id, (long) e->table,
MAYBE_TABLE_NAME(e->table))); MAYBE_TABLE_NAME(e->table)));
DBUG_RETURN(e->table); DBUG_RETURN(e->table);
} }
DBUG_PRINT("info", ("tid %lu is not mapped!", table_id)); DBUG_PRINT("info", ("tid %llu is not mapped!", table_id));
DBUG_RETURN(NULL); DBUG_RETURN(NULL);
} }
...@@ -102,10 +102,10 @@ int table_mapping::expand() ...@@ -102,10 +102,10 @@ int table_mapping::expand()
return 0; return 0;
} }
int table_mapping::set_table(ulong table_id, TABLE* table) int table_mapping::set_table(ulonglong table_id, TABLE* table)
{ {
DBUG_ENTER("table_mapping::set_table(ulong,TABLE*)"); DBUG_ENTER("table_mapping::set_table(ulong,TABLE*)");
DBUG_PRINT("enter", ("table_id: %lu table: 0x%lx (%s)", DBUG_PRINT("enter", ("table_id: %llu table: 0x%lx (%s)",
table_id, table_id,
(long) table, MAYBE_TABLE_NAME(table))); (long) table, MAYBE_TABLE_NAME(table)));
entry *e= find_entry(table_id); entry *e= find_entry(table_id);
...@@ -133,13 +133,13 @@ int table_mapping::set_table(ulong table_id, TABLE* table) ...@@ -133,13 +133,13 @@ int table_mapping::set_table(ulong table_id, TABLE* table)
DBUG_RETURN(ERR_MEMORY_ALLOCATION); DBUG_RETURN(ERR_MEMORY_ALLOCATION);
} }
DBUG_PRINT("info", ("tid %lu -> table 0x%lx (%s)", DBUG_PRINT("info", ("tid %llu -> table 0x%lx (%s)",
table_id, (long) e->table, table_id, (long) e->table,
MAYBE_TABLE_NAME(e->table))); MAYBE_TABLE_NAME(e->table)));
DBUG_RETURN(0); // All OK DBUG_RETURN(0); // All OK
} }
int table_mapping::remove_table(ulong table_id) int table_mapping::remove_table(ulonglong table_id)
{ {
entry *e= find_entry(table_id); entry *e= find_entry(table_id);
if (e) if (e)
......
...@@ -70,10 +70,10 @@ class table_mapping { ...@@ -70,10 +70,10 @@ class table_mapping {
table_mapping(); table_mapping();
~table_mapping(); ~table_mapping();
TABLE* get_table(ulong table_id); TABLE* get_table(ulonglong table_id);
int set_table(ulong table_id, TABLE* table); int set_table(ulonglong table_id, TABLE* table);
int remove_table(ulong table_id); int remove_table(ulonglong table_id);
void clear_tables(); void clear_tables();
ulong count() const { return m_table_ids.records; } ulong count() const { return m_table_ids.records; }
...@@ -83,14 +83,14 @@ class table_mapping { ...@@ -83,14 +83,14 @@ class table_mapping {
it, which only works for PODs) it, which only works for PODs)
*/ */
struct entry { struct entry {
ulong table_id; ulonglong table_id;
union { union {
TABLE *table; TABLE *table;
entry *next; entry *next;
}; };
}; };
entry *find_entry(ulong table_id) entry *find_entry(ulonglong table_id)
{ {
return (entry *) my_hash_search(&m_table_ids, return (entry *) my_hash_search(&m_table_ids,
(uchar*)&table_id, (uchar*)&table_id,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment