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

TcpDiscoverySpi: set connection check frequency to fixed value

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 2.5
    • 2.7
    • general

    Description

      Now connCheckFreq value calculated by some not obvious code:

      private void initConnectionCheckFrequency() {
                  if (spi.failureDetectionTimeoutEnabled())
                      connCheckThreshold = spi.failureDetectionTimeout();
                  else
                      connCheckThreshold = Math.min(spi.getSocketTimeout(), spi.metricsUpdateFreq);
      
                  for (int i = 3; i > 0; i--) {
                      connCheckFreq = connCheckThreshold / i;
      
                      if (connCheckFreq > 10)
                          break;
                  }
      
                  assert connCheckFreq > 0;
      
                  if (log.isInfoEnabled())
                      log.info("Connection check frequency is calculated: " + connCheckFreq);
              }
      

      It should be replaced with constant.

      Attachments

        Issue Links

          Activity

            People

              dkarachentsev Dmitry Karachentsev
              dkarachentsev Dmitry Karachentsev
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: