https://github.com/umbraco/Umbraco-CMS/blob/contrib/src/Umbraco.Core/DynamicRoot/QuerySteps/NearestDescendantOrSelfDynamicRootQueryStep.cs -->

_nodeFilterRepository.NearestDescendantOrSelfAsync

https://github.com/umbraco/Umbraco-CMS/blob/contrib/src/Umbraco.Infrastructure/Persistence/Repositories/Implement/DynamicRootRepository.cs#L71-L84 has a direct sql lookup? Is that bestpractice when adding a custom query step? Or should we be using the contentService if applicable?