Skip to content

Remove masked open_table_def - #613

Open
an3l wants to merge 1 commit into
MariaDB:10.3from
an3l:patch_create_table
Open

Remove masked open_table_def#613
an3l wants to merge 1 commit into
MariaDB:10.3from
an3l:patch_create_table

Conversation

@an3l

@an3l an3l commented Feb 13, 2018

Copy link
Copy Markdown
Contributor

In debugger, when "b my_message_sql" is used, when creating a new table (that doesn't exists) an error is raised, which is later ignored. This patch tries to prevent raising that error if we are in discovery mode.

@svoj svoj added this to the 10.3 milestone Feb 16, 2018
Comment thread sql/handler.cc Outdated
if (!hton)
flags|= GTS_NOLOCK;

<<<<<<< 2232784f0094ae7e81f5033fe8f3499e15224fed

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You didn't actually resolve the conflicts here! Let's go through this together.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, your changes updated libmariadb submodule. That shouldn't happen. Let's do a git tutorial next time we talk.

@an3l
an3l force-pushed the patch_create_table branch from 6856e38 to 0f1aba8 Compare February 21, 2018 23:57
@codership-jenkins

Copy link
Copy Markdown

Can one of the admins verify this patch?

@an3l
an3l force-pushed the patch_create_table branch from 0f1aba8 to 50b2179 Compare May 8, 2018 10:48
@CLAassistant

CLAassistant commented Mar 26, 2020

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@LinuxJedi LinuxJedi added the MariaDB Foundation Pull requests created by MariaDB Foundation label Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

MariaDB Foundation Pull requests created by MariaDB Foundation

Development

Successfully merging this pull request may close these issues.

6 participants