Details
-
Type:
Improvement
-
Status:
Open
-
Priority:
Minor
-
Resolution: Unresolved
-
Affects Version/s: 1.4.2
-
Fix Version/s: None
-
Component/s: None
-
Labels:None
-
Number of attachments :
Description
Everything functions correctly. However, there is one tweak I think we
should make. When you have multiple spans that cover the same entity,
we should only query one time.
Example:
A
/ \
B C
/ / \
D E F
Let's say we have an object graph like the one above that we want to
fill using Spans. We will pass our FetchSpecification to our
AFactory.Find method to retrieve the Entity.
If we use the following Spans {"C.F","C.E","B.D"}, this will give us the
correct data, but it will run the query for C twice. I propose that we
track which Entities we have already queried.
Chris