Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-3980

Redis-adapter redis connection is not reused when RedisTable is created

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.23.0
    • Component/s: None
    • Labels:
      None

      Description

      When creating RedisTable via ModelHandler, it relies on an instance method RedisEnumerator#deduceRowType to extract field type map, redis connection will be established and closed every time a table is created, which is unnecessary and redundant, so simply make the method a static one will solve the problem and make it more performant. Actually, CsvTable implementation works the right way.

        Attachments

          Activity
          $i18n.getText('security.level.explanation', $currentSelection) Viewable by All Users
          Cancel

            People

            • Assignee:
              Unassigned
              Reporter:
              neoremind neoremind

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

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

                  Issue deployment