[MDEV-26361] Spider: Use GROUP BY handler only when necessary and safe Created: 2021-08-13 Updated: 2022-05-29 Resolved: 2022-05-29 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Storage Engine - Spider |
| Fix Version/s: | N/A |
| Type: | Task | Priority: | Minor |
| Reporter: | Nayuta Yanagisawa (Inactive) | Assignee: | Nayuta Yanagisawa (Inactive) |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Issue Links: |
|
||||||||||||
| Epic Link: | Spider Refactoring | ||||||||||||
| Description |
|
In my opinion, Spider should not use GROUP BY handler unless it is necessary, for performance, and safe. That is because the GROUP BY handler reassembles the query from a general (possibly complex) execution plan, it tends to be more prone to mistakes than execution through the normal handler interface (which should result in a simpler query rassemblement). This is supported by the fact that many of the recently reported bugs are not reproducible on 10.2. I know that there are some problems with access via the handler interface (e.g., MDEV-26111), but I think it could be fixed. |
| Comments |
| Comment by Nayuta Yanagisawa (Inactive) [ 2021-08-15 ] |
|
The issue might not be necessary if |
| Comment by Nayuta Yanagisawa (Inactive) [ 2022-05-29 ] |
|
We resolve the issue in more elegant way in |