User tests: Successful: Unsuccessful:
Smart Search uses a MEMORY table to store intermediate results from the indexing process before permanently adding them to the index. When the intermediate data gets to big, the table is switched from MEMORY to a normal INNODB. At one point we standardized the default value to 7500 entries, but missed one spot. While the value is written automatically upon start of indexing, this cleans up that one default-value-fallback. It has no actual effect on the code execution, but is basically just some code-hygiene. That is why this simply requires a code review.
Since this only brings the default value in line with all other default values, this can only be done via code review.
Please select:
Documentation link for guide.joomla.org:
No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Administration com_finder |
Yes, that is still fine. Postgres doesn't have that issue and even for MySQL I'm looking if there is a better solution. But one step after the other.
I have tested this item ✅ successfully on 8eb0183
I have tested this item ✅ successfully on 8eb0183
I have tested this item ✅ successfully on 8eb0183
I have tested this item ✅ successfully on 8eb0183
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
PR-5.4-dev
|
||
RTC
RTC
| Labels |
Added:
RTC
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-07-31 20:34:27 |
| Closed_By | ⇒ | richard67 |
Thanks @Hackwar for this PR, and @brianteeman and @QuyTon for testing.
Just checking that this is still ok
joomla-cms/administrator/components/com_finder/src/Indexer/Indexer.php
Line 195 in 844e70e