Hide
added a comment -
Patch looks good, two possible improvements:
* ensureAuthorization seems to grab the rs metadata for nothing
* selectFidsSQL could just use the main select path with a query that grabs no attributes, using Query.NO_NAMES
Show
added a comment - Patch looks good, two possible improvements:
* ensureAuthorization seems to grab the rs metadata for nothing
* selectFidsSQL could just use the main select path with a query that grabs no attributes, using Query.NO_NAMES
* ensureAuthorization seems to grab the rs metadata for nothing
* selectFidsSQL could just use the main select path with a query that grabs no attributes, using Query.NO_NAMES