Uploaded image for project: 'Ignite'
  1. Ignite
  2. IGNITE-12800

SQL: local queries cursors must be closed or full read to unlock the GridH2Table.

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Critical
    • Resolution: Fixed
    • 2.8
    • 2.8.1
    • sql
    • None

    Description

      Root cause: local queries cursors must be closed or full read to unlock the GridH2Table.

      Proposal fix:

      • modify H2ResultSetIterator to use "paged mode": iterator reads N records into internal buffer and unlock the tables (similar to MapQueryResult; later we have to refactor these classes. They must use one code base to fetch data and lok/unlock tables)
      • modify the state logic of the QueryCursorImpl for lazy mode. Now the real query cancellation isn't called when result set is gathered. It is not valid for lazy mode.
      • add wrapper for iterator at the RegisteredQueryCursor because the state of query isn't tracked when results are read via Iterator at the client code.
      • fix tests that doesn't close query cursor for local queries.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            tledkov-gridgain Taras Ledkov
            tledkov-gridgain Taras Ledkov
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 20m
                20m

                Issue deployment