Indexing product (on save) should be done after committing the transaction

[tmotyl]

Right now while saving product, Magento indexes it in _afterSave() method.

https://github.com/magento/magento2/blob/master/app/code/core/Mage/Catalog/Model/Product.php line 516
Mage::getSingleton('Mage_Index_Model_Indexer')->processEntityAction( $this, self::ENTITY, Mage_Index_Model_Event::TYPE_SAVE );
The problem is that this method is called before committing save transaction.
see https://github.com/magento/magento2/blob/master/app/code/core/Mage/Core/Model/Abstract.php line 361

Current solution can lead some problems problems

• indexer might use old product data
• if indexer fails, e.g. throws exception, product will not be saved
• transaction stays open for the time of indexing
  -....

Instead, indexing product should be done in the event like 'catalog_product_model_save_commit_after'

[magento-team]

Hello. Thank you for reporting this issue. We are working on major changes in indexing functionality and we will take care of this one but this will take time.

[syedkazimi]

Hi @mage2-team,
When I import CSV products into Magento, with all the attributes, the layered navigation does not know about these products or their attributes (even after re-indexing many times).
Product or its attribute(s), e.g. Brand Name, only starts appearing under Brand Name filter after I MANUALLY EDIT PRODUCT, UPDATE ITS BRAND NAME ATTRIBUTE AND SAVE THE PRODUCT AGAIN (VIA ADMIN PANEL).
I have very large catalog of 300k products. So opening each product, updating its Brand Name attribute, and saving it again is not an option.
I have googled a lot, but did not find any specific answer for this problem.
Please Help me solving this issue.