bug: is_sorted should be true for SQLStream with replication keys

[pnadolny13]

Not sure if this is a bug or a design decision but shouldn't is_sorted be set to true in the default SQLStream implementation if we're adding an order by clause

sdk/singer_sdk/streams/sql.py

Line 192 in 6ba1519

query = query.order_by(replication_key_col)

?

Currently it accepts the default of False.

@edgarrmondragon what do you think?

[edgarrmondragon]

@pnadolny13 Yes, thanks for raising an actual issue 😅! I gave this a shot in tap-postgres a while ago (MeltanoLabs/tap-postgres#221) but doesn't seem to be as simple. We'd need to dive deeper.

[edgarrmondragon]

@pnadolny13 I have a PR for this: #1951. Let me know what you think.