Home - Waterfall Grid T-Grid Console Builders Recent Builds Buildslaves Changesources - JSON API - About

Console View


Categories: connectors experimental galera main
Legend:   Passed Failed Warnings Failed Again Running Exception Offline No data

connectors experimental galera main
fed2ad51323e...
fed2ad51323e3a741ca6638fd4aee8b333232f4e
Dave Gosselin
MDEV-39605: Assertion `!tbl->nested_join || tbl->sj_on_expr` failed

simplify_joins keeps a nested join intact when it contains
FULL JOIN tables and has siblings in its parent join list.
Flattening would let the optimizer interleave outside tables
between the FULL JOIN tables, which the null complement
algorithm cannot handle.  Such a nest carries neither on_expr
nor sj_on_expr and reaches table elimination, where we hit an
assertion which required one or the other.

Replace the assertion with a recursion into the nest.  Pass
on_expr=NULL so the nest itself is not eliminated, and set
all_eliminated to FALSE so an enclosing call does not eliminate
the parent.  Inner LEFT JOIN tables can still be eliminated by
the recursive call.
fc479d237a6e...
fc479d237a6e6e7a1abde1499edf58c144aa48f7
Vladislav Vaintroub
x
fafcaa81f983...
fafcaa81f9839684b8f98e0824568634b2960420
Daniel Bartholomew
bump the VERSION
f6adc063622d...
f6adc063622d7a83fd2c87b2a72b19818747bbbc
Dave Gosselin
Remove the unused AGGR_OP::flush_record method
f08889f1008c...
f08889f1008ce4e781bc397c8da186753c640e97
Dave Gosselin
MDEV-39715: semijoin nest under a FULL JOIN triggers assertion

When an IN-subquery in the ON clause of a JOIN to a FULL JOIN nest
becomes a semijoin, the new semijoin nest is placed inside the FULL JOIN
nest.  Normally that enclosing nest would be flattened, but a FULL
JOIN nest is not flattenable, so the semijoin nest remains.

check_interleaving_with_nj used to skip sj-nests when updating
counters, so every semijoin inner table also bumped the enclosing
nest's counter.  With two inner tables and two FULL JOIN tables, the
enclosing nest's counter went to 4 against a limit of 3 and the
assertion fired.

Count the semijoin nest as one child of its parent.  Bump the semijoin
nest's own counter on each semijoin inner table, and only bump the
parent once the semijoin nest is fully placed.  restore_prev_nj_state
is changed the same way for backtracking.
ed8404a73f93...
ed8404a73f93e79aff7227bb4560a44cbfdb5d1c
Hemant Dangi
MDEV-39721: wsrep_notify.cc: reject shell-unsafe characters in joiner-supplied member fields

Issue:
wsrep_notify_status() interpolated members[i].name() (the peer's
wsrep_node_name) and members[i].incoming() verbatim into a command
string that is then executed via 'sh -c' by wsp::process. A peer
joining the cluster with shell metacharacters in its wsrep_node_name
or wsrep_node_incoming_address caused arbitrary commands to run on
every cluster member that had wsrep_notify_cmd configured.
MDEV-39413 introduced safe() for the same threat class in the SST
scripts but did not cover the C++ wsrep_notify path.

Solution:
Validate each substituted field against a narrow whitelist before
interpolating. Node name allows alnum and -_. ; node address
additionally allows :[]/ for host:port and [ipv6] forms. On bad
input the notification is skipped with an error log instead of
forwarding the unsafe value to sh -c.
e860cac5faa3...
e860cac5faa31298b16468984f22cb0267538ac1
Andrei Elkin
High priority MDL_BACKUP_BLOCK_COMMIT_RPL

When the standard request of MDL_BACKUP_COMMIT is not granted to a worker
it first searches for a possible grantee that is a parallel slave
worker and if there exist one with greater commit order the request
type is escalate to the highest which is re-submitted.

To the changes in sql_base.cc,
note there also exist a request of the highest (new) type at time of
Xid_apply_log_event::do_record_gtid which is a committing phase of
slave transaction when it is executing an implicit statement.
It must be able to bypass any waiting lock by backup process even
without yet knowing whether the transaction is going to be enrolled
into a pending to commit slave group. When later it turns out
not the case the strong MDL_STATEMENT requested lock would've been
already relinquished.

Tested with previous commit's  refined and extended BASE's
  rpl.parallel_backup, rpl.rpl_parallel_backup_worker_retry
and new added
  rpl.rpl_parallel_backup_waits_worker_retry

Todo:
According to Svoj's review, this patch still needs
removing the slave specific code from the MDL.
That is changes in MDL_context::acquire_lock().

Probably
ha_commit_trans() would call first
    try_acquire_lock_impl()
when it returns empty ticket a part of the added to acquire_lock
Teammate-Aware Escalation Check would be invoked just to compute
`has_teammate` and when that is true to call the whole BASE version
of MDL_context::acquire_lock() having the highest type of request.
d960ced40116...
d960ced40116d64111267d355a84c38fdabbc755
Vladislav Vaintroub
1
d8dd579dd01e...
d8dd579dd01e6a62ee02454d41936fdecab81057
Vladislav Vaintroub
x
d5dd63fd035d...
d5dd63fd035d14cddb690bfe8ac40bef0c5673ca
Dave Gosselin
MDEV-39713: Propagate dep_tables for FULL JOIN with constant ON

A FULL JOIN with a constant ON expression did not encode
the outer table dependency in the on_expr because the
constant references no tables.  The enclosing nest's
dep_tables, however, still carries that dependency, so
propagate it.
c8a145708e2c...
c8a145708e2c92e54dd30e6411247229db074080
Dave Gosselin
MDEV-39663: Assertion `found_tables > 0' failed

Early on in FULL OUTER JOIN development, it seemed prudent to
attach the ON condition to both tables during FULL OUTER JOIN
parsing, because either table in the join could be considered
the inner or outer table, depending on perspective.  This
decision ended up complicating both simplify_joins and the
FULL JOIN rewrites to other join types, as well as dependent
table propagation during make_join_statistics.  Dependent
table propagation was not correct because both tables in the
FULL JOIN carried the ON expression which confused
make_join_statistics.

We could update make_join_statistics to propagate dependent
tables for both sides of a join, but that's essentially changing
the mainstream join processing path for the sake of one case
case.  Instead, just stop attaching the ON condition to both
tables in a FULL OUTER JOIN, which simplifies the FULL JOIN
rewrites and relies on pre-existing table dependency propagation
in make_join_statistics.
c04473cd77cf...
c04473cd77cff10ee43f89fa7e929f1091146ea9
Dave Gosselin
MDEV-39619: Compute FULL JOIN nest tables recursively

compute_full_join_nest_tables built the bitmap of tables that must
stay adjacent in the join order by iterating leaf_tables and OR'ing in
each enclosing nest's direct_children_map.  That missed FULL JOIN
tables that are themselves a nest of inner joins.  With those tables
missing from full_join_nest_tables, the adjacency restriction in
get_allowed_nj_tables let the optimizer descend into an inner nest
whose remaining tables depended on a table outside it, triggering the
found_tables>0 assertion in best_extension_by_limited_search.

Replace the leaf walk with a recursive walk over top_join_list.  For
every TABLE_LIST flagged JOIN_TYPE_FULL, OR in its
nested_join->used_tables (for a nest) or table->map (for a base
table), and recurse into every nested_join so a FULL JOIN table that
is itself a nest contributes all of its leaf tables.
b4a80422bfee...
b4a80422bfeec93079a430c080fffbda8f6fa574
Sergei Golubchik
Merge branch '10.11' into 11.4
b2050fdb4a87...
b2050fdb4a8776422baf01a41bf86845994edb97
Sergei Golubchik
strengthen safe() in wsrep_sst_common, just in case
9b9bcdee2e76...
9b9bcdee2e76305c8bf4e682b83eaa83bf6539de
Dave Gosselin
MDEV-39569: Skip FULL JOIN rewrite to inner side of an outer join

Prevent simplify_joins from rewriting a chained FULL JOIN into a query
where a FULL JOIN could end up on the inner side of another outer
join.  Of course, this means that we will have a null complement pass
that the rewritten query would have avoided.  Once we support FULL
JOINs on the inner side of outer joins, in phase 3, then we can relax
this constraint.
9a52192c0b51...
9a52192c0b518b0b5455bafe3906606ff8a73514
Vladislav Vaintroub
MDEV-39719 Fix memory allocation errors on Windows ARM64 CI

Reduce the default max size from 8TB to 128GB. on ARM64, allocating
multiple huge areas (like CI does, for multiple processes) can result in
exhaustion of internal kernel memory. Presumably non-paged pool, used
for page translation tables (this info is provided by AI, can't verify).
93049da77c62...
93049da77c62158b7f591fb668ec1c9d1e4725bd
Vladislav Vaintroub
y
9126c6319a41...
9126c6319a41d1e0704be70d8c49668edc04cf02
Dave Gosselin
MDEV-39618:  Assertion failed

In run_fj_null_complement_pass, the FULL JOIN null complement pass
called ha_end_keyread() on the right table to switch from index access
before rescanning, but never restored keyread after it finished; so
do that.
90b029f2c9f6...
90b029f2c9f6d065267a825eac087d8cde20b618
ParadoxV5
MDEV-6589: Remove `have_debug_sync` requirement

The test for `rpl.rpl_parallel_mdev6589` required
`include/have_debug_sync.inc` but did not use `@@debug_sync`
(besides RESETting it during cleanup). It uses `MASTER_GTID_WAIT()`.
8f29dce6eb80...
8f29dce6eb804b85e389f3f159045847a0c13cb5
Dave Gosselin
In end_send, grab the list of fields from the prior join_tab if available

Allocate the full_join_duplicate_filter with operator new.  Add a comment
to the assertion and condense a couple of conditionals together.
8c007d6bf807...
8c007d6bf8070c803e2e5b7ce4652a3d5f29aa19
Vladislav Vaintroub
x
7d086be762a0...
7d086be762a08c8d34e8fcb03814cf2222d03860
Vladislav Vaintroub
x
78bc473197eb...
78bc473197eb1b6f0b39cccb422e3c628d406b8b
Dave Gosselin
FULL JOIN, straight_join and optimizer hint support

Previously the FULL JOIN tables had to come first in the join
order, before any other table.  They still have to stay next to
each other, but they can now appear before, after, or between
other tables in the FROM clause.

Update the JOIN_ORDER, JOIN_PREFIX, and JOIN_SUFFIX hint conflict
check to match.  The new check only rejects a hint when it would force
a non-FULL JOIN table to sit between two FULL JOIN tables, which is
the case that actually breaks the null-complement rescan.  It finds
this by computing the set of predecessors of FULL JOIN tables and the
set of successors of FULL JOIN tables, after the hint's dependencies
have been added, and rejects the hint when those sets overlap on a
non-FULL-JOIN, non constant table.
77582ad7175d...
77582ad7175d55dd02ae2b2ef924506939bb7b96
Vladislav Vaintroub
x
6fb907b46437...
6fb907b46437bee84535b18df7c7146a5d842a06
bsrikanth-mariadb
MDEV-39412: parse error reading tabs in ranges

Note:
while reading from information_schema.optimizer_context one level of unescaping
is already done i.e. (\\t becomes \t or \\\\t becomes \\t)

w.r.t the MDEV, there are 2 problems: -

1.
When reading from the sql script file, json parser is not able to parse
the range value in json_read_value() from json_lib.c
"ranges": [
            "(b\t\t\t\t\t\t) <= (b) <= (b???????)"
          ],
mainly the \t\t stuff, and hence a warning.
It also stops loading the context into memory.
Since, a new table is created with empty data, and without context,
we get Impossible WHERE noticed after reading const tables

2.
There is unescaping call being made in read_string() from sql_json_lib.cc
while parsing of the context. With this \\t was becoming \t.
However, print_range() from opt_range.cc already does escaping of the values.
The value "b\t\t\t" was in fact produced as "\b\\t\\t\\t".
Later, we try to compare range values from the query and the context.

Here a mismatch is found because, in one case there is escaping,
and in the other case escaping got removed.

Solution
========
Since, there are 2 levels of unescaping being performed, 1. during sql
parse of the context from information_schema, and 2. during
read_string. So, we need to have 2 levels of escaping.

First is done in the dump_mrr_info_calls() - here
json_escape_to_string() is used.

Second is done at the end of store_optimizer_context(), for the entire
opt_context. Here a newly introduced function escape_json_for_sql_literal()
is used which does escaping only for backslash, and single quote.
6b35f1748d64...
6b35f1748d64858d21fc1ca76976bd56483d5c63
Sergei Golubchik
Merge branch '11.4' into 11.8
6676a3caaa93...
6676a3caaa930e37b2888065f595e1f8b51c2160
Dave Gosselin
Make FULL JOIN work correctly when other tables sit beside it

A FULL JOIN produces its result in two stages.  The first stage walks
the joined tables like a LEFT JOIN, emitting every left row paired
with its matches on the right (or with NULLs if there were none).  The
second stage walks the right table again and emits the right rows that
never found a match, padded with NULLs on the left.

Until now the second stage ran exactly once per query, after the whole
first stage had finished.  That worked only when the FULL JOIN was the
entire query.  As soon as another table sat outside the FULL JOIN, the
result became wrong, because the "right rows with no matches" needed
to be paired with the outer table's rows just like the matched rows
were.  To avoid producing wrong answers, the optimizer was forced to
put FULL JOIN tables first.

This change moves the second stage so that it runs as part of the same
nested loop that produced the first stage's matches.  Each time the
outer loop advances to a new row, the right table is rescanned for its
unmatched rows, and those rows flow through the rest of the query
exactly the way ordinary join output does.

With the two stages now interleaved, the optimizer no longer has to
force the FULL JOIN to the front of the join order.  Tables outside
the FULL JOIN can sit on either side of it, and the FULL JOIN itself
only needs to stay together as one contiguous group.
606610c1e57e...
606610c1e57eac7394e6c4409e8b09620a896914
Dave Gosselin
MDEV-39607:  Segmentation fault in alloc_full_join_duplicate_filters

The count of JOIN_TAB instances passed to alloc_full_join_duplicate_filters
was the total count, including the constant tables.  But start_tab points
to the first JOIN_TAB after the constant tables, so it was walking off the
end of the JOIN_TAB instances in memory.  Just take the starting point from
the JOIN directly and include the constant tables (which we'll need to do
in phase 3 anyway).
57e4a63a8e32...
57e4a63a8e327baacc205d1b6d7577e4a9af3cab
Oleksandr Byelkin
make resalt the same for different platforms
4b75caf71326...
4b75caf71326a6bf910ce1618065e0c3e9f6fa0f
ParadoxV5
Fix operator typo in `||` chain

There is a `,` breaking the chain; it will make the sub-chain
after it ignore any failure from the sub-chain before it.

This change was originally **unsolicitedly** suggested by Gemini;
that is, I did not activate Gemini nor provide any prompt,
but rather the CI-triggered Gemini comment came to me like junk mail.
By verifying the bot output and creating this patch independently
*without AI assistance*, I confirm I understand the change and
can claim its authorship and responsibility,
and *I remain have never provided AI-generated code*.

Co-authored-by: gemini-code-assist <[email protected]>
46a8eb42a520...
46a8eb42a520193686d9a16d4cea4b3e002917e4
Sergei Golubchik
fix plugins.feedback_os_release failure on sles-1600

just the comment at the beginning of its /etc/os-release
is already more than 256 bytes
418ae57946b0...
418ae57946b033f331617c94857d510d1f63a6d6
Dave Gosselin
RelWithDebInfo builds in buildbot could fail

The crash happened because we computed an incorrect stopping point
for the JOIN_TAB walk when finding the left-most JOIN_TAB for a
FULL JOIN in a bush child.  When inspecting bush children, the
eligible JOIN_TABs are those of the particular bush, so walk over
those.  Previously, we took the end point as JOIN::join_tab, which
was incorrect, causing the walk to go into arbitrary memory.
3d10c7065560...
3d10c7065560468de5248032aa41db8b20caa104
Vladislav Vaintroub
x
2464556cd455...
2464556cd45543c9ce580761584fdc40904272d6
ParadoxV5
MDEV-20122: Remove `Master_info::is_demotion`

MDEV-20122 introduced this field, but it was never actually used.
`CHANGE MASTER TO master_demote_to_slave=…` only uses the parser
field `LEX_MASTER_INFO::is_demotion_opt` and is not recorded in
either `@@master_info_file` or `@@relay_log_info_file`,
as it takes effect during CHANGE MASTER rather than during replication.
233e284b458a...
233e284b458adf6fcdf19797341c10d90bdef3ed
Dave Gosselin
MDEV-38508: Constant table detection

If a table that's in a FULL OUTER JOIN is found to be a const
table, then don't allow the constant table optimization to
take place.

Later, when we support FULL OUTER JOIN on the inner side of
other join types then we may be able to relax this restriction.
1cb9cbd88f4b...
1cb9cbd88f4bc3104c3feb011632ff0d62eb3cd6
Vladislav Vaintroub
1
197f92bee02d...
197f92bee02d8e836f529f37625be69b83e7acbd
Sergei Golubchik
Merge branch '10.6' into 10.11
185f91d80d85...
185f91d80d85a8a23c7eb8f34ced95dd0083a463
ParadoxV5
MDEV-6589: Remove `have_debug_sync` requirement

The test for `rpl.rpl_parallel_mdev6589` required
`include/have_debug_sync.inc` but did not use `@@debug_sync`
(besides RESETting it during cleanup). It uses `MASTER_GTID_WAIT()`.
09d471f48ef0...
09d471f48ef09e8165eae7e29cb4c428b7482067
Dave Gosselin
all table names, aliases, view names, etc are now lowercase
BuildBot (0.8.9) working for the MariaDB project.
Page built: Sun 24 May 2026 23:41:51 (UTC)