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

Query timeout redesign

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 2.8.1
    • 2.10
    • sql
    • None
    • Adds ability to set up default query timeout for the all nodes of the cluster
    • Docs Required, Release Notes Required

    Description

      Motivation:
      Now the query timeout is set up for each node separately by the node configuration. This property isn't propagated for the all nodes of cluster. Also user cannot change / disable query timeout without restart all nodes of the cluster.

      Proposal fix:

      • Adds the default query timeout property to the DistributedSqlConfiguration. Use distributed metastore to store and manage the property.
      • Deprecates the SqlConfiguration#defaultQueryTimeout property and use it to set up initial value of new property DistributedSqlConfiguration#defaultQueryTimeout
      • Adds info about explicit query timeout to GridH2QueryRequest (boolean flag explicitTimeout=false by default). This is necessary so that the default timeout may be used for queries from old nodes.
      • When query timeout is set to zero by old node (explicitTimeout=false) we assume this is the default value and use default timeout for this queries.

      Attachments

        Issue Links

          Activity

            People

              tledkov-gridgain Taras Ledkov
              tledkov-gridgain Taras Ledkov
              Votes:
              0 Vote for this issue
              Watchers:
              3 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 - 40m
                  40m