- Issue created by @mkalkbrenner
- Merge request !238Issue #3526572 by mkalkbrenner: Provide information about request time when indexing starts → (Merged) created by mkalkbrenner
- 🇦🇹Austria drunken monkey Vienna, Austria
drunken monkey → made their first commit to this issue’s fork.
- 🇦🇹Austria drunken monkey Vienna, Austria
Looks good, and seems small enough of a change to be acceptable even if it probably won’t be useful to more than a few people/modules.
I made some minir changes to the MR, please review. Also the newly adapted test is failing, my guess would be that the index instance use in the indexing batch is not the same as in the test method.
If you can think of another quick way to test the new method, that would be great, but otherwise I’d say it’s also no big deal if there is no test coverage, as long as you’re confident it works as desired (as the main user of the new method) and think it’s unlikely it will ever break due to some changes in Core or our batch indexing code. -
drunken monkey →
committed c478b2ff on 8.x-1.x authored by
mkalkbrenner →
Issue #3526572 by mkalkbrenner, drunken monkey: Added request time...
-
drunken monkey →
committed c478b2ff on 8.x-1.x authored by
mkalkbrenner →