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
I ran into a error when executing a SELECT * from a somewhat bigger table. While the regular table browse function uses some form of paging to limit the result set to be returned, it's probably not appropriate to fiddle with user-supplied queries in the same way, even if it's a blunt a SELECT *, so let's maybe detect if the returned error is a one as in the snippet below and return something nicer such as "Query result set is too big. Please use a TOP clause or change the statement to retrieve fewer rows"
A change to the server-side API (Atelier API v6) to limit the number of rows to return is on its way to 2023.1, due for preview release in the next month or so. Taking advantage of this on the SQLTools side should significantly lower the risk of running into this error.
Atelier API v6 adds support for a new max query parameter that accepts a positive integer and restricts the number of rows returned for each result set to that value. If the value is not a positive integer or the parameter was not passed, all rows will be returned. This extension should expose a setting that controls this parameter for all requests and the default should probably be between 500-1000.
I ran into a error when executing a SELECT * from a somewhat bigger table. While the regular table browse function uses some form of paging to limit the result set to be returned, it's probably not appropriate to fiddle with user-supplied queries in the same way, even if it's a blunt a SELECT *, so let's maybe detect if the returned error is a one as in the snippet below and return something nicer such as "Query result set is too big. Please use a TOP clause or change the statement to retrieve fewer rows"
sqltools-intersystems-driver/src/ls/irisdb.ts
Line 141 in bf8d99c
The text was updated successfully, but these errors were encountered: