problem in SQLServerStatementSQLVisitor.class

[JINWANDALAOHU250]

Bug Report

Which version of ShardingSphere did you use?

 <dependency>
  <groupId>org.apache.shardingsphere</groupId>
  <artifactId>shardingsphere-jdbc-core</artifactId>
  <version>5.3.2</version>
</dependency>

Which project did you use? ShardingSphere-JDBC or ShardingSphere-Proxy?

ShardingSphere-JDBC

Expected behavior

execute sql like

SELECT
    * 
FROM
    message_mst 
WHERE
    readed = 0 
ORDER BY
    msg.create_dt DESC OFFSET ? ROWS FETCH NEXT ? ROWS ONLY

then parse to

Actual SQL: slave :::
SELECT
    * 
FROM
    message_mst 
WHERE
    readed = 0 
ORDER BY
    msg.create_dt DESC OFFSET ? ROWS FETCH NEXT ? ROWS ONLY

::: [10, 5]

Actual behavior

### Cause: java.sql.SQLException: Unknown exception: Index: 2, Size: 2
; uncategorized SQLException for SQL []; SQL state [HY000]; error code [30000]; Unknown exception: Index: 2, Size: 2; nested exception is java.sql.SQLException: Unknown exception: Index: 2, Size: 2

Reason analyze (If you can)

In class SQLServerStatementSQLVisitor.class#visitOrderBy ,offset paramIndex and rowcount paramIndex not right, caused PaginationContext get param in List

[terrymanu]

No plan to enhance with SQLServer for now, does any one want to do it?

[JINWANDALAOHU250]

Is it that I want to be simple, I think it should only take parameterMarkerSegments.size() - 1 to solve this problem

[terrymanu]

I cannot suggest without investigate. Please try.