You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In #1626 (comment), we found that we rely on MainQueryResultProcessor.getRequestedValuesFromResultsWithFullGraphPattern to prevent extra incoming links in a query like this:
Here we get four regions on different pages in the same book. Without MainQueryResultProcessor.getRequestedValuesFromResultsWithFullGraphPattern, all the links from all pages to the book show up as incoming links each time the book appears as a dependent resource in the response to the API request. This is because ConstructResponseUtilV2.splitMainResourcesAndValueRdfData doesn't consider the response tree structure when it starts constructing link values for incoming links. It would be better to handle this logic in ConstructResponseUtilV2.splitMainResourcesAndValueRdfData; then we could get rid of MainQueryResultProcessor.
The text was updated successfully, but these errors were encountered:
In #1626 (comment), we found that we rely on
MainQueryResultProcessor.getRequestedValuesFromResultsWithFullGraphPattern
to prevent extra incoming links in a query like this:Here we get four regions on different pages in the same book. Without
MainQueryResultProcessor.getRequestedValuesFromResultsWithFullGraphPattern
, all the links from all pages to the book show up as incoming links each time the book appears as a dependent resource in the response to the API request. This is becauseConstructResponseUtilV2.splitMainResourcesAndValueRdfData
doesn't consider the response tree structure when it starts constructing link values for incoming links. It would be better to handle this logic inConstructResponseUtilV2.splitMainResourcesAndValueRdfData
; then we could get rid ofMainQueryResultProcessor
.The text was updated successfully, but these errors were encountered: