Fix OData filtering by a base class member #2387
Conversation
| var queryModel = NhRelinqQueryParser.Parse(_expression); | ||
| // Use the modified TransparentIdentifierRemovingExpressionVisitor in order to support expressions that were created | ||
| // programmatically and do not mimic what the C# compiler generates. | ||
| queryModel.TransformExpressions(TransparentIdentifierRemovingExpressionVisitor.ReplaceTransparentIdentifiers); |
There was a problem hiding this comment.
Fortunately that is cached in the query plan cache, so it is acceptable to traverse it again I think.
There was a problem hiding this comment.
I agree, the only thing that we could do is to make the capacity of the query cache configurable as currently is hardcoded to 128
which may be a little to small for larger applications. Of course, this should be done in a separate PR.
There was a problem hiding this comment.
Of course, this should be done in a separate PR.
Especially since that could be a deep rabbit hole. I think we should instead get rid of this SoftLimitMRUCache and use something like System.Runtime.MemoryCache.
The limit is hardcoded to 128, but that is not a strong limit. SoftLimitMRUCache uses only weak references past 128 entries and lets the garbage collector decides whether to free them or not.
3 participants
Fixes #2380