Github user patricker commented on the issue:
https://github.com/apache/nifi/pull/1510
@stanislavprokopov Thoughts on how I can distinguish that this adapter is
only for MS SQL v2012 or greater? Right now Database Adapters don't have a
description, but I could add a field, populate it for all of them, and show it
on the Adapter drop down list.
I don't understand how I could get around setting ORDER BY when just using
TOP, assuming I will have additional pages of data. Can you show me an example
of what Page 1 (TOP) and Page 2 (OFFSET) queries would look like such that they
will work together?
If you meant that there will be no paging and only TOP will be used, then
the code already supports this without an ORDER BY:
```
if (limit != null && offset == null){
query.append("TOP ");
query.append(limit);
query.append(" ");
}
```
The ORDER BY check only kicks in if `offset` also does not equal `null`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
