Compatibility with WooCommerce HPOS

[felipeelia]

Is your enhancement related to a problem? Please describe.

This issue is to investigate ElasticPress's compatibility with WooCommerce High-Performance Order Storage.

My findings so far:

• 100% compatibility would be very hard to achieve, as data requests to the new custom table are spread throughout the code base, like here, for example;
• It seems the best candidate for a replacement with a compatibility layer would be the OrdersTableQuery class, as it is responsible for running queries
• The OrdersTableQuery class is called directly by the OrdersTableDataStore class here, so we might need to override that one too.
• Due to the way the WC container works, it won't allow us to simply replace a class, so we might need a new filter to be added.

Code of Conduct

• I agree to follow this project's Code of Conduct

[vedanshujain]

Hey @felipeelia , from WC team here. I would love to help you with HPOS compatibility for ElasticPress. It looks like currently you hook into posts_pre_query to hijack WP query and populate results from elasticsearch, would be happy to add similar filter for HPOS as well. It seems we won't need to create new index for HPOS orders, since there is already a allowlist for what data can be synced, and we can add HPOS orders in the same index (note that we always insert a shop_order_placehold post and use the same ID in order tables, so a ID conflict won't happen).

Let us know if there is anything else you would need besides an equivalent posts_pre_query filter.

[felipeelia]

@vedanshujain you got it right. It is worth saying though that an equivalent filter for posts_pre_query would be a good start, but we will probably need some further changes, especially given the number of times the code seems to be touching the database directly.

[vedanshujain]

Cool cool, we will get a PR started for posts_pre_query. Please let us know what you discover during development, we would be happy to add more action/filters where they makes sense, and especially if a similar filter already exists with the posts.

[vedanshujain]

@felipeelia , I can try and give it a shot for a draft PR, if this is still on the roadmap?