[GitHub] [fineract] BLasan edited a comment on pull request #1598: Removed unwanted result-set previous method

[GitBox <gi...@apache.org>]

BLasan edited a comment on pull request #1598:
URL: https://github.com/apache/fineract/pull/1598#issuecomment-773227196


   > Hi @BLasan - no, the ResultSet initially doesn't point at the first row. But this is not a question about the initial position: the same ResultSet is being iterated by both the calling function and the called function. The calling function expects the ResultSet to be returned in such state that by calling rs.next() it gets the next record to be processed. If you remove rs.previous() from the called function, that will no longer be the case and one record gets skipped. You can try it out by stepping through the code with a debugger and seeing what happens (as long as your JDBC driver supports rs.previous())
   
   Hello @ptuomola , could you please mention the calling function of the extractData with regarding this issue? I was unable to find it as I'm new to the community. Could you please help me in finding this? :)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org