A normal catalog structure is like this: you have a few high level categories under the catalog, then each high level category has a few lower level categories under it, then each lower level category has their children, so on and so forth until you reach the leaves – catalog entries.
However it is not uncommon that you have multiple children (categories and entries) directly under catalog. Even though that is not something you should do, it happens.
But that is not without drawbacks. You might notice it is slow to route to a product. It might not be visible to naked eyes, but if you use some decent profilers (which I personally recommend dotTrace), it can be fairly obvious that your site is not routing optimally.
Why?
To route to a specific catalog content, for example http://commerceref/en/fashion/mens/mens-shirts/p-39101253/
, the default router have to figure out which content is mapped to an url segment. So with default registration where the catalog root is the default routing root, we will start with the catalog which maps to the first part of route (fashion
). How do it figure out which content to route for the next part (mens
) ?
Until recently, what it does it to call GetChildren
on the catalog ContentReference
. Now you can see the problem. Even with a cached result, that is still too much – GetChildren
with a big number of children is definitely expensive.
We noticed this behavior, thanks to Erik Norberg. An improvement have been made in Commerce 12.10 to make sure even with a number of children directly under Catalog, the router should perform adequately efficient.
If you can’t upgrade to 12.10 or later (you should!), then you might have a workaround that improve the performance. By adding your own implementation of HierarchicalCatalogPartialRouter
, you can override how you would get the children content – by using a more lightweight method (GetBySegment
)
public class CustomHierarchicalCatalogPartialRouter : HierarchicalCatalogPartialRouter
{
private readonly IContentLoader _contentLoader;
public CustomHierarchicalCatalogPartialRouter(Func<ContentReference> routeStartingPoint, CatalogContentBase commerceRoot, bool enableOutgoingSeoUri) : base(routeStartingPoint, commerceRoot, enableOutgoingSeoUri)
{
}
public CustomHierarchicalCatalogPartialRouter(Func<ContentReference> routeStartingPoint, CatalogContentBase commerceRoot, bool supportSeoUri, IContentLoader contentLoader, IRoutingSegmentLoader routingSegmentLoader, IContentVersionRepository contentVersionRepository, IUrlSegmentRouter urlSegmentRouter, IContentLanguageSettingsHandler contentLanguageSettingsHandler, ServiceAccessor<HttpContextBase> httpContextAccessor) : base(routeStartingPoint, commerceRoot, supportSeoUri, contentLoader, routingSegmentLoader, contentVersionRepository, urlSegmentRouter, contentLanguageSettingsHandler, httpContextAccessor)
{
_contentLoader = contentLoader;
}
protected override CatalogContentBase FindNextContentInSegmentPair(CatalogContentBase catalogContent, SegmentPair segmentPair, SegmentContext segmentContext, CultureInfo cultureInfo)
{
return _contentLoader.GetBySegment(catalogContent.ContentLink, segmentPair.Next, cultureInfo) as CatalogContentBase;
}
}
And then instead of using CatalogRouteHelper.
MapDefaultHierarchialRouter , you register your router directly
var referenceConverter = ServiceLocator.Current.GetInstance<ReferenceConverter>();
var contentLoader = ServiceLocator.Current.GetInstance<IContentLoader>();
var commerceRootContent = contentLoader.Get<CatalogContentBase>(referenceConverter.GetRootLink());
routes.RegisterPartialRouter(new HierarchicalCatalogPartialRouter(startingPoint, commerceRootContent, enableOutgoingSeoUri));
(ServiceLocator
is just to make it easier to understand the code. You should do this in an IInitializationModule
, so use context.Locate.Advanced
instead.
This is applicable from 9.2.0 and newer versions.
Moral of the story:
- Catalog structure can play a big role when it comes to performance.
- You should do profiling whenever you can
- We do that too, and we make sure to include improvements in later versions, so keeping your website up to date is a good way to tune performance.