Details
-
Bug
-
Status: Closed (View Workflow)
-
Major
-
Resolution: Fixed
-
10.2(EOL)
-
None
Description
--source include/have_partition.inc
|
|
|
INSTALL SONAME 'ha_rocksdb'; |
CREATE TABLE t1 (a INT) ENGINE=RocksDB PARTITION BY HASH(a) PARTITIONS 2; |
INSERT INTO t1 (a) VALUES (1),(2); |
ALTER TABLE t1 ADD PARTITION PARTITIONS 2; |
--source include/restart_mysqld.inc
|
SELECT 1; |
|
10.2 0992be927e1c686c39c39fe53fc2a7869d55143d |
mysqld: /data/src/10.2/sql/mysqld.cc:2160: void mysqld_exit(int): Assertion `global_status_var.global_memory_used == 0' failed.
|
170622 19:22:06 [ERROR] mysqld got signal 6 ;
|
|
|
#3 <signal handler called>
|
#4 0x00007fca74f0ffcf in raise () from /lib/x86_64-linux-gnu/libc.so.6
|
#5 0x00007fca74f113fa in abort () from /lib/x86_64-linux-gnu/libc.so.6
|
#6 0x00007fca74f08e37 in __assert_fail_base () from /lib/x86_64-linux-gnu/libc.so.6
|
#7 0x00007fca74f08ee2 in __assert_fail () from /lib/x86_64-linux-gnu/libc.so.6
|
#8 0x000055fa47a9713c in mysqld_exit (exit_code=0) at /data/src/10.2/sql/mysqld.cc:2160
|
#9 0x000055fa47a9efa5 in mysqld_main (argc=129, argv=0x55fa49ea9f50) at /data/src/10.2/sql/mysqld.cc:6079
|
#10 0x000055fa47a937b0 in main (argc=7, argv=0x7fff875cab38) at /data/src/10.2/sql/main.cc:25
|
Attachments
Activity
Could you do it from ha_partition::close() or from ha_partition::~ha_partition()? Both are called (in that order) after mysql_rename_partitions() at the end of fast_alter_partition_table().
serg, thanks for the reply!
> Could you do it from ha_partition::close()
Cannot do that from there, as I wrote above, alter_close_table() calls ha_partition->close(), and then ha_partition::rename_partitions() requires that newly created ha_xxx ojbects are still present.
> or from ha_partition::~ha_partition()?
This one seems to work, created a patch with this, now testing in buildbot.
> Both are called (in that order) after mysql_rename_partitions() at the end of fast_alter_partition_table().
Looking at the above, I dont see where exactly ha_partition should delete its newly-created child partitions. The workflow is apparently tightly coupled with the SQL layer.
Should we add handler::partitioning_ddl_done() call and have SQL layer call it, and ha_partition use this method?
(serg Any thoughts?)