Uploaded image for project: 'Spark'
  1. Spark
  2. SPARK-44920

Use await() instead of awaitUninterruptibly() in TransportClientFactory.createClient()

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.3.3, 3.4.2, 3.5.0
    • 3.4.2, 3.5.0, 4.0.0, 3.3.4
    • Spark Core
    • None

    Description

      This is a follow up for SPARK-44241:

      That call added an `awaitUninterruptibly()` call, which I think should be a plain `await()` instead. This will prevent issues when cancelling tasks with hanging network connections. 

      This issue is similar to SPARK-19529

      Attachments

        Issue Links

          is caused by

          Bug - A problem which impairs or prevents the functions of the product. SPARK-44241 Set io.connectionTimeout/connectionCreationTimeout to zero or negative will cause executor incessantes cons/destructions

          • Major - Major loss of function.
          • Resolved
          relates to

          Bug - A problem which impairs or prevents the functions of the product. SPARK-19529 TransportClientFactory.createClient() shouldn't call awaitUninterruptibly()

          • Major - Major loss of function.
          • Resolved
          links to

          Pull request #42619 [Github] Pull Request #42619 (JoshRosen)

          Activity

            People

              joshrosen Josh Rosen
              joshrosen Josh Rosen
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: