[MDEV-16326] Warning 4094 has a very confusing message Created: 2018-05-30 Updated: 2023-04-27 |
|
| Status: | Confirmed |
| Project: | MariaDB Server |
| Component/s: | Server |
| Affects Version/s: | 10.3, 10.4, 10.5, 10.6, 10.7 |
| Fix Version/s: | 10.4, 10.5, 10.6 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Federico Razzoli | Assignee: | Rucha Deodhar |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | warning | ||
| Description |
|
When warning 4094 occurs, I see messages like this: Note (Code 4094): At line 8 in test.test0 This makes the warning confusing and not helpful. To add confusion, this warning is not listed in the relevant documentation pages:
I'm not adding a test case because you already have several test cases where 4094 is supposed to occur. |
| Comments |
| Comment by Federico Razzoli [ 2018-05-30 ] | |||||||||
|
I have a strong feeling that I'm seeing this warning far too many times when testing stored aggregate functions. But without an error message it's hard for me to say if the bug is in MariaDB or in my tests. So I'll wait for a fix here. | |||||||||
| Comment by Sergei Golubchik [ 2018-06-21 ] | |||||||||
|
It's ER_SP_STACK_TRACE. The error message is, precisely, "At line %u in %s". As far as I can see from the tests, it happens as
The message is pretty clear if you know what it means, but I agree that on itself it's quite puzzling. We need at least to document it, but also, perhaps, to reword to be less confusing. | |||||||||
| Comment by Sergei Golubchik [ 2018-06-21 ] | |||||||||
|
What about "Error %u at line %u in %s"? It'd be, say
| |||||||||
| Comment by Elena Stepanova [ 2018-07-19 ] | |||||||||
|
I guess it's as good as anything. The current one is confusing because it does look like something is missing. | |||||||||
| Comment by Federico Razzoli [ 2018-07-19 ] | |||||||||
|
Time passed and I cannot find out easily. But actually you already pointed out the problem: if you know what it means, it's clear... otherwise it means nothing. Documenting it would make it more useful, and would make clear that the "poor" message is not a bug. | |||||||||
| Comment by Elena Stepanova [ 2018-09-04 ] | |||||||||
|
I assume we agreed on changing the error message as above. | |||||||||
| Comment by Jörg Schierstein [ 2020-02-22 ] | |||||||||
|
/*-- Testcase for DELIMITER ;; | |||||||||
| Comment by Elena Stepanova [ 2021-10-15 ] | |||||||||
|
serg, rucha174, |