Uploaded image for project: 'MariaDB Connector/J'
  1. MariaDB Connector/J
  2. CONJ-436

ResultSet first() implementation

Details

    Description

      Hello,

      we have upgraded our JDBC driver to mariadb driver from mysql driver and it has caused an unforeseen issue.

      https://github.com/MariaDB/mariadb-connector-j/blob/master/src/main/java/org/mariadb/jdbc/internal/queryresults/resultset/MariaSelectResultSet.java

      The SQLException is thrown but we think the SQLException should NOT be thrown if the rowPointer is already at 0 - because in this case, there is no backward movement.

      This has not been an issue with our mysql jdbc driver.

      I appreciate that the JDBC interface docs may not be clear on this, but i think it makes sense not to throw an exception if rowPointer == 0

      Attachments

        Activity

          Ascending order - Click to sort in descending order
          zsolt.janos Zsolt Janos created issue -
          zsolt.janos Zsolt Janos made changes -
          Field Original Value New Value
          Description Hello,

          we have upgraded our JDBC driver to mariadb driver from mysql driver and it has caused an unforeseen issue.

          https://github.com/MariaDB/mariadb-connector-j/blob/master/src/main/java/org/mariadb/jdbc/internal/queryresults/resultset/MariaSelectResultSet.java

          The SQLException is thrown but we think the SQLException should NOT be thrown if the rowPointer is already at 0 - because in this case, there is no backward movement.

          This has not been an issue with our mysql or postgres jdbc driver.

          I appreciate that the JDBC interface docs may not be clear on this, but i think it makes sense not to throw an exception if rowPointer == 0           		Hello,

          we have upgraded our JDBC driver to mariadb driver from mysql driver and it has caused an unforeseen issue.

          https://github.com/MariaDB/mariadb-connector-j/blob/master/src/main/java/org/mariadb/jdbc/internal/queryresults/resultset/MariaSelectResultSet.java

          The SQLException is thrown but we think the SQLException should NOT be thrown if the rowPointer is already at 0 - because in this case, there is no backward movement.

          This has not been an issue with our mysql jdbc driver.

          I appreciate that the JDBC interface docs may not be clear on this, but i think it makes sense not to throw an exception if rowPointer == 0
          diego dupin Diego Dupin added a comment - - edited

          Hi zsolt.janos

          Right, JDBC indicate that absolute positionning must throw an exception when scroll type is TYPE_FORWARD_ONLY.
          But in reallity, there is no reason for that when not in streaming, (setFetchSize set).

          Will be done on next corrective version.

          diego dupin Diego Dupin added a comment - - edited Hi zsolt.janos Right, JDBC indicate that absolute positionning must throw an exception when scroll type is TYPE_FORWARD_ONLY. But in reallity, there is no reason for that when not in streaming, (setFetchSize set). Will be done on next corrective version.
          diego dupin Diego Dupin made changes -
          Status Open [ 1 ] Confirmed [ 10101 ]
          diego dupin Diego Dupin made changes -
          Fix Version/s 1.5.9 [ 22516 ]
          diego dupin Diego Dupin made changes -
          Resolution Fixed [ 1 ]
          Status Confirmed [ 10101 ] Closed [ 6 ]
          serg Sergei Golubchik made changes -
          Workflow MariaDB v3 [ 79742 ] MariaDB v4 [ 134958 ]

          People

            diego dupin Diego Dupin
            zsolt.janos Zsolt Janos
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Git Integration

                Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.