Can not fetch an object from Cursor when a RowBounds is used

[kazuki43zoo]

MyBatis version

3.4.0

Database vendor and version

Any

Test case or example project

See CursorSimpleTest#testCursorWithRowBound.

Steps to reproduce

I added foreach processing for Cursor in test case as follow:

Cursor<User> usersCursor = sqlSession.selectCursor("getAllUsers", null, new RowBounds(1, 2));
for(User user: usersCursor) { // Add
    System.out.println(user);
}

Expected result

I expected print as follow:

User{id=2, name='User2'}
User{id=3, name='User3'}

Actual result

This code occurred the CursorException.

User{id=2, name='User2'}

org.apache.ibatis.cursor.CursorException: Cursor is closed

    at org.apache.ibatis.cursor.defaults.DefaultCursor$CursorIterator.next(DefaultCursor.java:207)
    at org.apache.ibatis.submitted.cursor_simple.CursorSimpleTest.testCursorWithRowBound2(CursorSimpleTest.java:169)
...

Related links

• Cursor implementation #437

[emacarron]

Looks like RowBounds and a Cursor are alternative methods so I would say it is Ok that rowbounds does not work. Anyway we can either make it work or throw a more gentle exception.

edit: Sorry I have just checked the case. Seems it is working well. But the Cursor is not expected to be used that way.

[kazuki43zoo]

The following code will work (without the RowBounds).

Cursor<User> usersCursor = sqlSession.selectCursor("getAllUsers", null);
for(User user: usersCursor) {
    System.out.println(user);
}

I think this issue is bug.
I think should be reconsider a solution to support the RowBounds on Cursor fetching.

What do you think ?

[emacarron]

I do agree @kazuki43zoo

[kazuki43zoo]

Hi @gdarmont , can you help us for this issue ?

[gdarmont]

Hi @kazuki43zoo and @emacarron.
I've just made a pull request that fixes the problem.