Hi, Nikita,
See, e.g. handler::is_clustering_key() or handler::ha_rnd_pos_by_record()
On May 28, Nikita Malyavin wrote:
> On Wed, 24 May 2023 at 18:32, Sergei Golubchik wrote:
>
> > > +bool handler::is_root_handler() const
> > > +{
> > > + return this == tab
rtainly leaf" one by comparing it with ha_resolve_by_name outputs
> for "myisam" and "innodb".
>
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://launchpad.net/~maria-developers
Po
not, could you share it, please?
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://launchpad.net/~maria-developers
Post to : maria-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~maria
ot; cpu with 2000 bogomips.
if you'll simply hack num_cpus() to return 1, it'll fail later on
something like $self->{cpus}[0]->{bogomips}
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://lau
gt; --- a/storage/myisam/ha_myisam.cc
> +++ b/storage/myisam/ha_myisam.cc
> @@ -983,6 +983,7 @@ int ha_myisam::close(void)
>
> int ha_myisam::write_row(const uchar *buf)
> {
> + DBUG_ASSERT(is_root_handler() || table->file->ht != ht);
>
> +
> +drop table t;
lots of unvisible spaces at line ends, please remove them
> +
> +--echo # MDEV-31172 Server crash or ASAN errors in online_alter_check_autoinc
> +create table t (a int, b int, c char(8), key(a,b,c));
> +alter table
hild(fk.delete_method)
> +|| fk_modifies_child(fk.update_method))
> +{
> + reason= "Tables with CASCADE/SET NULL foreign keys are";
> + goto unsupported;
> +}
> + }
> +
> + if (!online_alter_check_autoinc(thd, alter_info, table))
> + {
> +reason=
y.
Right, it's also future proof, no more zlib issues ever.
Users will get a smaller source download too.
bindled zlib is only needed for our bintar builds, never for debs.
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
_
at zlib detection is part of cmake, not something we've changed,
so it's could be caused by a change in your build environment.
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://launchpad.net/~maria-develo
t;
> Alternatively, we could introduce some bool handler::root_handler just
> now, without waiting for a demand, but I'm afraid it could become
> another source for bugs. So better later on demand.
>
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
_
Hi, Nikita,
On May 08, Nikita Malyavin wrote:
> On Fri, 5 May 2023 at 20:46, Sergei Golubchik wrote:
>
> > > #ifdef HAVE_REPLICATION
> > > - if (unlikely(!error && table->s->online_alter_binlog))
> > > + if (unlikely(!error && table-
: "not ",
> - (*field_ptr)->field_name.str,
> - (long) (field_ptr - begin_ptr)));
> + dbg->field_name.str,
> + (long) ()));
eh? you surely meant `(long) fidx` ?
how did it ev
else
add // trans engine, end of statement
> +{
> + DBUG_ASSERT(!is_ending_transaction);
> + cache.store_prev_position();
> +}
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://launchpa
to filter out individual partitions and have your
binlog_log_row_online_alter called only for the main ha_partition
handler.
> error= binlog_log_row_online_alter(table, before_record, after_record,
> log_func);
> #endif // HAVE_REPLICATION
Regards,
Sergei
();
> }
> -else if (!is_ending_transaction)
> +else if (!commit) // rollback
> +{
> cache.restore_prev_position();
> +}
> +else
add // trans engine, end of statement
> +{
> + DBUG_ASSERT(!is_ending_transaction);
> + cache.store_prev_p
> +}
> -Item *LEX::make_item_func_substr(THD *thd, Item *a, Item *b, Item *c)
> -{
> - return (thd->variables.sql_mode & MODE_ORACLE) ?
> -new (thd->mem_root) Item_func_substr_oracle(thd, a, b, c) :
> -new (thd->mem_root) Item_func_substr(thd, a, b, c);
>
Hi, Alexander,
On Mar 10, Alexander Barkov wrote:
> On 3/7/23 2:20 PM, Sergei Golubchik wrote:
> > I'm sorry it took a while.
> >
> > I'm still thinking about the whole thing, it's a rather big change
> > for a really fringe functionality. But I failed to come up with
> +*/
> +if (my_time_tmp &&
> +(my_time_tmp != TIMESTAMP_MAX_VALUE ||
> + error_code != ER_WARN_DATA_OUT_OF_RANGE))
why `my_time_tmp != TIMESTAMP_MAX_VALUE` ? shouldn't error_code check
be sufficient?
>to_tz->gmt_sec_to_TIME(ltime, m
he bit, like
used_default_coll_mapping |= 1 << i;
and then, again, print affected collations, if any. Most often
used_default_coll_mapping will likely be zero
one more question. In, say, 10.10->11.1 replication
master and slave will have different default collations, but
thd-&g
ude
> +#endif
> +
> #ifdef __cplusplus
> extern "C" {
> #endif
>
> #ifndef MYSQL_ABI_CHECK
why not to put #include here?
> #ifdef _WIN32
> -#include
> #ifndef __cplusplus
> #define inline __inline
> #endif
> -#else
> -#include
I s
RPM_client_PACKAGE_CONFLICTS
> - "MariaDB-server < 10.6.0")
> + "MariaDB-server < 11.0.0")
because of resolve-stack-dump, ok
> SETA(CPACK_RPM_common_PACKAGE_CONFLICTS
> - "MariaDB-server < 10.6.1")
> + "MariaDB-server < 11.0.0"
(or discussed,
if you disagree) before pushing.
Other comments apply to the old spider code that this patch moved. If
you'd like you can say "I'm just moving spider code around" and ignore
all comments of that kind.
/Sergei
On Jan 22, Yuchen Pei wrote:
> commit dcd69cbd19a
> Author: Yu
;open_options|= extra_open_options;
> -result_code= admin_recreate_table(thd, table);
> +result_code= admin_recreate_table(thd, table) ? HA_ADMIN_FAILED :
> +HA_ADMIN_OK;
good catch
> thd->open_options&=
Hi, Sergei,
On Jan 13, Sergei Petrunia wrote:
> revision-id: 0bef50e50b5 (mariadb-10.4.27-33-g0bef50e50b5)
> parent(s): 5db970fc760
> author: Sergei Petrunia
> committer: Sergei Petrunia
> timestamp: 2023-01-04 16:50:12 +0300
> message:
>
> MDEV-20501: Assertion `m
;
>}
> - Item_func_hash *hash_item= new(mem_root)Item_func_hash(thd,
> *field_list);
> +
> + Item_func_hash *hash_item= make_unique_hash_func(thd, mem_root,
> +
> table->s->mysql_version,
> +
ape_item(THD *thd, Item *escape_item,
> String *tmp_str,
> if (escape_str)
> {
>const char *escape_str_ptr= escape_str->ptr();
> - if (escape_used_in_parsing && (
> - (((thd->variables.sql_mode & MODE_NO_BACKSLASH_ESCAPES)
10);
> ---error ER_CANT_CREATE_GEOMETRY_OBJECT
> +--error ER_CANT_CREATE_GEOMETRY_OBJECT,ER_INTERNAL_ERROR
> select st_distance_sphere(x'010300040004',
> multipoint(point(124,204)), 10);
> ---error ER_CANT_CREATE_GEOMETRY_OBJECT
> +--error ER_CANT_CREATE_GEOMETRY_OBJECT,ER_I
t; +SELECT name, body_utf8, body FROM mysql.proc WHERE name like 'sp1';
> +
> +SET GLOBAL event_scheduler=ON;
> +
> +SELECT name, body_utf8, body FROM mysql.event;
> +CREATE EVENT ev1 ON SCHEDULE EVERY 1 SECOND DO INSERT INTO t1 VALUES (1,
> 'abc');
> +SELECT name, body_
);
> if (unlikely($$ == NULL))
>MYSQL_YYABORT;
>}
> -| expr comp_op predicate %prec '='
> +| boolean_test comp_op predicate %prec '='
>{
> $$= (*$2)(0)->create(thd, $1, $3);
> if (unlikely($$ == NULL))
>
e whole commit is about fixing the test?
How can I see that the test works?
> diff --git
> a/mysql-test/suite/galera/r/galera_sst_rsync_innodb__innodb_dir,debug.rdiff
> b/mysql-test/suite/galera/r/galera_sst_rsync_innodb__innodb_dir,debug.rdiff
> new file mode 100644
> index 000
; usr/bin/mariadb-backup
> > > usr/bin/mbstream
> > > -usr/share/man/man1/mariabackup.1
>
> but this man link is removed.
why did you remove this man symlink? I thought Debian is pretty strict
on having manpages for everything
> > you forgot to move symlinks to man pages
>
ty requirements).
I suggest to keep those symlinks in the mariadb-test package.
Your other PR will add a warning. And eventually we drop them.
Optional/Recommended/Suggested - sounds like too much toubles
for a *test* package.
> diff --git a
%' to database 'some_db'
> +show tables from some_db;
> +ERROR 42000: Access denied for user 'foo'@'%' to database 'some_db'
> +#
> +# bar user has access to the database (no denies present)
> +#
> +connect con2,localhost,bar,,;
> +use some_db;
> +show tables from some_db;
Hi Rex,
Please find below review input for the collection of patches for these MDEVs.
First, please try to have each piece of functionality in its own commit.
You can use "git rebase -i" to make your last commits be one commit,
as well as re-order them.
Then, you can do a carefully considered
Hi, Nikita,
On Nov 01, Nikita Malyavin wrote:
> On Mon, 31 Oct 2022 at 21:22, Sergei Golubchik wrote:
>
> > > I hope RPL_TABLE_LIST can be preserved between the events. Is it
> > > so?
> >
> > Yes, it appears to be created in Table_map_log_event and removed
Hi, Nikita,
On Oct 31, Nikita Malyavin wrote:
> On Mon, 31 Oct 2022 at 14:08, Sergei Golubchik wrote:
> > On Oct 31, Nikita Malyavin wrote:
> > >
> > > > Also, I've looked at your latest branch. What were you
> > > > optimizing with the commit 3afa3288
e same table and all events have
full rows. There is no need to recalculate the key per event.
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://launchpad.net/~maria-developers
Post to : maria-develo
Hi, Nikita,
On Oct 18, Nikita Malyavin wrote:
> On Tue, 18 Oct 2022 at 01:01, Sergei Golubchik wrote:
>
> > > > > diff --git a/sql/log_event_server.cc b/sql/log_event_server.cc
> > > > > index 422d496d5c3..25705d13641 100644
> > > > >
ter partitions and change column data types at the same
> + time.
> +*/
> +Create_field::upgrade_data_types(alter_info->create_list);
> + }
> +
>if (create_info->check_fields(thd, alter_info,
> table_list->table_name, table_list
tes m_curr_row_end */
> + error= unpack_current_row(rgi, _cols_ai);
> + m_table->default_field= default_field;
> +
> + if (unlikely(error))
> goto err;
>
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Ma
ith a working server, I do not think
> that our server will survive if something out of the server will
> change frms.
May be not, but it doesn't mean we need to introduce intentionally more
places that can cause a crash. What will happen if you would not set
share->tabledef_version.length= 0
ct per-segment
> flags that were calculated during the table CREATE time, no matter
> what the old (CREATE time) and new (ALTER TIME) per-index compression
> flags are, and no matter if they are equal or not.
>
> All other engine ignore key compression flags, so this change
> is saf
Hi, Oleksandr,
On Oct 19, Oleksandr Byelkin wrote:
> On Wed, Oct 19, 2022 at 5:09 PM Sergei Golubchik wrote:
> > On Oct 19, Oleksandr Byelkin wrote:
> > >
> > >if (share->view_def->parse((uchar *) >tabledef_version, NULL,
> > >
r, both of the same
TABLE_SHARE?
> return TRUE;
> + }
>DBUG_ASSERT(share->tabledef_version.length ==
> MICROSECOND_TIMESTAMP_BUFFER_SIZE-1);
> +
>return FALSE;
> }
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
_
Hi, Nikita,
On Oct 17, Nikita Malyavin wrote:
> Hello Sergei!
>
> On Tue, 4 Oct 2022 at 21:02, Sergei Golubchik wrote:
>
> > Hi, Nikita,
> >
> > This review applies to the combined diff e2f8dff^..52f489e
> >
> > On Oct 04, Nikita Malyavin wrote:
&g
goto ok_or_after_trg_err;
>}
> + if (table->versioned(VERS_TIMESTAMP) && table->vers_write)
> + {
> +store_record(table, record[2]);
> +error = vers_insert_history_row(table);
if you do that, you shouldn't have been doin
elongs?
> + mysql_mutex_lock(>LOCK_share);
>
>if (!(share->tabledef_version.str=
> (uchar*) alloc_root(>mem_root,
> MICROSECOND_TIMESTAMP_BUFFER_SIZE)))
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
_
Hi, Marko,
On Oct 10, Marko Mäkelä wrote:
> On Sun, Oct 9, 2022 at 2:03 PM Sergei Golubchik wrote:
> > I've spent more time on this issue, and it looks like there is no
> > structure in InnoDB with the life time till the end of the statement
> > (or, at least, nothing easil
Hi, Nikita,
On Sep 01, Sergei Golubchik wrote:
>
> > This new solution simply does the following:
> > * Sets up a referenced table list in TABLE instance (sql_base.cc)
> > * Iterates through it along with dict_table_t::referenced_set
> > (row_upd_check_reference
Hi, Nikita,
On Sep 27, Nikita Malyavin wrote:
> Sergei, I have reworded the commit message, please see it here:
> https://github.com/MariaDB/server/commit/3a3064e355bac20ed56ae807e790068e16dd16f3
Same thing. I still cannot understand from the comment what the problem
was.
>> > Th
ask) != (new_key->flags & key_flag_mask))
> return Compare_keys::NotEqual;
>
> - if ((table_key->flags & HA_KEYFLAG_MASK) !=
> - (new_key->flags & HA_KEYFLAG_MASK))
> + if (table_key->algorithm != new_key->algorithm)
> return Compare_ke
))
>continue;
>
> -tmp= key->rec_per_key[last_part];
> +tmp= key->rec_per_key[parts_suit - 1];
> if (best_key_nr == MAX_KEY || (tmp > 0 && tmp < best_rec_per_key))
> {
>best_key_nr= i;
> best_key= key;
>bes
(tabledef_version.length && ...
that is, if m_table_ref_version matches, then you don't need to do any
further checks, the table is fine. If m_table_ref_version differs,
meaning, the table was reopened, then you check tabledef_version and
triggers.
Perhaps the simplest fix would be to remove r
> +MICROSECOND_TIMESTAMP_BUFFER_SIZE)))
> +return TRUE;
> + share->tabledef_version.length= 0; // safety if the drfinition file is
> brocken
>
> + DBUG_ASSERT(share->view_def != NULL);
> + if (share->view_def->parse((uchar *) >table
_time_to_time, hr_time_from_time
> >
> > What do you mean by "made for events" ?
>
> It request val field from passed object (I mean the macro), and I am
> not sure if rewriting that macro a good idea.
it works with my_hrtime_t values, so if you store high-resolutio
ues.
> > > +timestamp.second_part= trigger->ms_create_time % 100;
> >
> > hrtime_to_time() and hrtime_sec_part()
>
> they made for events, so I made hr_time_to_time, hr_time_from_time
What do you mean by "made for events" ?
Regards,
Sergei
VP of MariaDB
t; + /* all grants must be revoked from this role by now. propagate this */
> + propagate_role_grants(acl_role, PRIVS_TO_MERGE::ALL);
> +
>my_hash_delete(_roles, (uchar*) acl_role);
>DBUG_RETURN(1);
> }
>
Regards,
Sergei
VP of MariaDB Server Engine
ot;No AUTH_GSSAPI plugin" unless $ENV{AUTH_GSSAPI_SO};
> +return "No AUTH_GSSAPI plugin" unless ($ENV{AUTH_GSSAPI_SO} or ($^O eq
> 'MSWin32'));
The second condition normally is something like
or $::mysqld_variables{gssapi} eq "ON";
>
> retu
p:
>if (p_table)
> @@ -10041,6 +10046,8 @@ static int handle_grant_data(THD *thd, Grant_tables&
> tables, bool drop,
>}
>
>/* Handle stored routines table. */
> + if (tables.procs_priv_table().table_exists())
> + {
This is ok, but also, please, check that a
set()->name);
>}
Note that in 10.10 you need to fix json_table code
inside a big if() condition:
...
(!(m_explicit_cs->state & MY_CS_PRIMARY) &&
(str->append(STRING_WITH_LEN(" COLLATE ")) ||
str->append(_explicit_cs->
mal patch to show the fix, practically
I'd accompany it with a small cleanup patch)
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
On Sep 02, Andrei Elkin wrote:
>
> > Query_log_event cannot just randomly use some
> > options_written_to_bi
event
has to remember options_written_to_bin_log itself, there's not much we
can do about it.
Or it can apply the mask in the constructor and store not flags2 but
the new value of thd->variables.option_bits.
Or it can extend flags2 to cove
prebuilt, I mean.
I couldn't find it (it's a complex patch, so I couldn've missed it).
As far as I can see, it's only used to store a pointer to TABLE.
So it seems to me than a simpler fix for this bug could be:
* remove vc_templ caching (mysql_table and mysql_table_query_id)
* store TABLE* in upd_n
, so the replica node
> remains unaware of it.
>
> The solution would be to guess on the replica side on the mode used by
> master. To make a deduction reliable, master's locked_tables_mode state
> is passed for query log events.
Regards,
Sergei
VP
ity
>if (get_flags(STMT_END_F))
> status_var_increment(thd->status_var.com_stat[SQLCOM_DELETE]);
>
> + const uchar *curr_row_end= m_curr_row_end;
> + unpack_row(rgi, m_table, m_width, m_curr_row, _cols,
> + _row_
op database testdb;
> +
> +--echo #
> +--echo # bug with automatically added PUBLIC role
> +--echo #
> +
> +--echo # automaticly added PUBLIC
> +delete from mysql.global_priv where user="PUBLIC";
> +flush privileges;
> +GRANT SELECT on test.* to PUBLIC;
> +
>
e: remove test and test_* database hacks in the test
> suite
let's say, it's ok.
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://launchpad.net/~maria-developers
Post to : maria-developers@list
modify mysqltest in a way that would
avoid the need of changing tests.
> connection con0;
> select hex(@a);
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://launchpad.net/~maria-developers
P
# NOTE: having row_start=0 might be useful and can mean
> # "there is no information on when the history was started" (an opposite to
> row_end=MAX_TIMESTAMP)
>
> Maybe we should allow it? Just to make the user not invent some values
> for this purpose (like
Hi, Alexander,
> > -ERROR HY000: Illegal parameter data types inet6 and int for operation 'SET'
> > +ERROR HY000: Unknown CAST(int AS inet6) in assignment of 'target'
sorry, forgot to add, let's also add a db/table name here? and use
the proper backtick quoting for identifiers
Reg
HERE COLUMN_NAME='source'
> AND TABLE_NAME='t1'
> @@ -47,6 +52,7 @@ INSERT INTO t3 VALUES
> (2,
> (SELECT source FROM t2 ORDER BY id LIMIT 1),
> (SELECT source FROM t2 ORDER BY id LIMIT 1));
> +ERROR 22003: Cannot get geometry object from data you send to t
false;
> +}
> my_error(ER_ILLEGAL_PARAMETER_DATA_TYPES2_FOR_OPERATION, MYF(0),
> type_handler()->name().ptr(), from->name().ptr(), "SET");
> return true;
if you wouldn't want to change the error message in a followup commit,
I'd suggested somethin
Hi, Alexander,
On Jul 29, Alexander Barkov wrote:
>Hello Sergei,
>
> I have a couple of suggestions:
>
> > commit 7b8304045272111a6f4d44196d6b37cbfef06f37
> > Author: Sergei Golubchik
> > Date: Wed Jul 20 17:31:48 2022 +0200
&g
Hi, Alexander,
I added tests and created an MDEV.
Specific replies below:
On Jul 29, Alexander Barkov wrote:
>Hello Sergei,
>
> > commit 5a362d486b30fdaf3c7a360737331767154b4ee8
> > Author: Sergei Golubchik
> > Date: Mon Jul 18 22:53:27 2022 +0200
> >
Hi, Nikita,
Sure, as you like
On Jul 13, Nikita Malyavin wrote:
> I did, in rpl_alter_extra_persistent.test.
>
> Btw, maybe also rename this file, into something not referring to
> PERSISTENT?
>
> On Wed, 13 Jul 2022 at 23:41, Sergei Golubchik wrote:
>
> > Hi,
Hi, Nikita,
Please, add a test case for DEFAULT in replication, something like I
described below.
On Jul 13, Nikita Malyavin wrote:
> On Thu, 7 Jul 2022 at 16:05, Sergei Golubchik wrote:
>
> > Hi, Nikita,
> >
> > This is good, but I think fill_extra_persistent_columns
ot delete it! Even though MERGE tables do not have their children
> @@ -11854,8 +11857,11 @@ copy_data_between_tables(THD *thd, TABLE *from,
> TABLE *to,
>
> // We'll be filling from->record[0] from row events
> bitmap_set_all(from->write_set);
> -//
t know the code. I'd suggest to add method, like
struct Rpl_table_data
{
...
bool is_online_alter() { return copy_fields != NULL; }
...
};
btw, it fits very well in your Rpl_table_data simplification.
> + | MODE_NO_AUTO_VALUE_ON_ZERO;
Regards,
Serg
ns
> - */
> - if (unlikely(error= fill_extra_persistent_columns(table, cols->n_bits)))
> -DBUG_RETURN(error);
> -
>/*
> We should now have read all the null bytes, otherwise something is
> really wrong.
Regards,
Sergei
VP of MariaDB Server Engineering
>
> With regards,
>
> Martijn Tonies
> Upscene Productions
> https://www.upscene.com
>
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://launchpad.net/~maria-developers
Post to
> MDEV-27265 Improve contraction performance in UCA collations
>
> Adding a hash table for contractions.
>
> The old code iterated through all items in MY_CONTRACTIONS,
> and was much slower, especially for those contractions
> in the end of the list.
>
Regards,
Serg
t; +*/
> >
> > if it's generated, do you need to check it in?
> > perhaps it should be generated during the build?
> > you've checked in allkeys1400.txt anyway.
>
> Right, we can consider it.
>
> Btw, I've checked it all versions:
>
> $ ls mysql-test/st
+ }
> + void restore_binlog_vars(my_bool _log_bin)
this doesn't need to be a reference, just a normal pass-by-value will
do.
> + {
> +do_log_bin= thd->variables.sql_log_bin;
> +thd->variables.sql_log_bin= sav_log_bin;
> +thd->set_binlog_bit();
> + }
>
Hi, Sergei,
On May 04, Sergei Petrunia wrote:
> revision-id: 4ba5b444b4b (mariadb-10.2.43-97-g4ba5b444b4b)
> parent(s): 70555454b4c
> author: Sergei Petrunia
> committer: Sergei Petrunia
> timestamp: 2022-05-04 12:24:48 +0300
> message:
>
> MDEV-28437: Assert
Hi, Sergei,
The solutions looks fine
Two comments below, about the implementation.
ok to push after addressing them
On May 03, Sergei Petrunia wrote:
> revision-id: 85cc56875e9 (mariadb-10.2.43-75-g85cc56875e9)
> parent(s): 3b6c04f44c4
> author: Sergei Petrunia
> committer: Ser
Hi, Rucha,
Looks good. please also check that row_number values match what's in a
warning, and then ok to push!
On Apr 26, Rucha Deodhar wrote:
> On Tue, Apr 26, 2022 at 12:41 AM Sergei Golubchik wrote:
> > what if you combine it with union?
> > create table t (a char(1)) v
)->lim.get_select_limit())
>break;
> -int rc=
> - result->send_data_with_check(*elem, sl->master_unit(), send_records);
> +rc= result->send_data_with_check(*elem, sl->master_unit(), send_records);
you don't need to do that anymore (it was needed
> + my_printf_error(ER_INVALID_ROLE, "User %`s@%`s has not been
> granted role %`s",
> + MYF(0), user, host, rolename);
>}
>else
>{
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
__
e.
But whether you move `int res` declaration inside or not - it's ok to
push, no need to ask for another review.
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
On Apr 25, Rucha Deodhar wrote:
> revision-id: 5f8ffc7271a (mariadb-10.2.40-251-g5f8ffc7271a)
> p
Hi, Aleksey,
ok to push, thanks!
On Apr 20, Aleksey Midenkov wrote:
> Sergei,
>
> I updated the branch accordingly. There is your patch fa444975d0f
> where I added log_current_statement().
>
> On Wed, Apr 20, 2022 at 1:12 PM Sergei Golubchik wrote:
> >
> > H
Hi, Aleksey,
Okay, sure.
I was going to push it after the release.
And in 10.3 to simplify future merges.
On Apr 20, Aleksey Midenkov wrote:
> Hi Sergei!
>
> On Tue, Apr 19, 2022 at 11:13 AM Sergei Golubchik wrote:
> >
> > Hi, Aleksey,
> >
> >
gt;
> Where is your patch?
It's really just renaming, almost nothing else.
But here it is, attached.
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
commit 0593fd1926f
Author: Sergei Golubchik
Date: Mon Apr 11 22:06:21 2022 +0200
cleanup: log_curre
> +*/
> +next_state= MY_LEX_IDENT_OR_QUALIFIED_SPECIAL_FUNC;
I don't understand why you did it in the lexer and not in the parser, like
- | REPLACE '(' expr ',' expr ',' expr ')'
+ | opt_schema REPLACE '(' expr ',' expr ',' expr '
binlog (or binlog trx
cache) even if it otherwise wouldn't be.
So, two flags. Clear names and semantics. "vers_created_partitions" fits
into the second use case, so one should use OPTION_BINLOG_THIS_STMT for it.
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mar
ay.
But I have a fix for this different bug already, in bb-10.3-serg-MDEV-24176
So, go ahead and push, then I'll merge with 10.2 and push my fix too.
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://laun
rk-in-progress" commit that removes
init_lex_with_single_table and CONTEXT_ANALYSIS_ONLY_VCOL_EXPR.
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
___
Mailing list: https://launchpad.net/~maria-developers
Post to
NLY_VCOL_EXPR
(it's set inside init_lex_with_single_table()) is wrong, what you're
doing is not "context analysys only", you're preparing items for
evaluation.
Let's wait for Sanja to close his MDEV-25638, and then I'll check this
test case again.
Regards,
Sergei
VP of MariaDB Server Eng
ode(thd, killed_status == NOT_KILLED);
> +StatementBinlog stmt_binlog(thd,
> thd->binlog_need_stmt_format(trans_table));
> res= thd->binlog_query(THD::ROW_QUERY_TYPE,
> thd->query(), thd->query_length(),
> t
==
with the commit comment saying that "on_table_fill_finished() should
always be done at the end of open(), even if result is not
Select_materialize, but, for example, Select_fetch_into_spvars"
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.org
ist);
> +
> + if (table->pos_in_table_list->security_ctx)
> thd->security_ctx= tl->security_ctx;
> - bool res= fix_session_vcol_expr(thd, vcol);
> +
> + inited= true;
> + return false;
> +}
> +
Regards,
Sergei
VP of MariaDB Server Engineering
and secur...@mariadb.
1 - 100 of 2336 matches
Mail list logo