-
Notifications
You must be signed in to change notification settings - Fork 9.4k
#26206 Add information about currently reindexed index. #26207
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#26206 Add information about currently reindexed index. #26207
Conversation
Hi @lbajsarowicz. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
…ge() in foreach()
b89d577
to
79fe786
Compare
Clean The integration test is - in general - useless, written wrong way, so I'm not sure about it's corectness. |
42c65e5
to
f67a576
Compare
88455bd
to
5bdadc0
Compare
#26225 Has to be merged first (!). |
Hi @pmclain, thank you for the review.
|
✔️ QA Passed |
Hi @lbajsarowicz, thank you for your contribution! |
Description (*)
Answers the need for more information about failed reindex.
Change additionally improves performance by replacing
foreach() { array_merge() }
witharray_merge(...$args)
. Additionally with-v
debug information included about exception trace.Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
N/A
Contribution checklist (*)