Fix pylint false positives for ThreadPoolExecutor

[srikaaviya]

Description

This PR changes the concurrent.futures import style in three files to bypass a known no-name-in-module pylint parsing bug introduced when adding Python 3.14 support.

By using import concurrent.futures and fully qualifying concurrent.futures.ThreadPoolExecutor and concurrent.futures.Future, we cleanly resolve the false positive without needing any # pylint: disable comments or global config overrides.

Fixes #4199

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

The official tox linting suites passed successfully:

Does This PR Require a Core Repo Change?

No.

Checklist:

• Followed the style guidelines of this project
• Changelogs - (N/A - purely internal linting fix)
• Unit tests - (N/A - fixing lint false positive)
• Documentation - (N/A)

[xrmx]

I don't think this is the right way to fix this. We should fix all the issue we have with a newer pylint version and then remove the comments.

[DylanRussell]

do we need this commented out ? can we just remove it ? here and everywhere else ?

[JWinermaSplunk]

I agree with @xrmx in that we should bump the pylint version instead of using the fully qualified names.

[srikaaviya]

@xrmx @DylanRussell Thanks for the feedback! I've reverted the import workarounds and bumped pylint to 4.0.5 in dev-requirements.txt which resolves the false positives natively on Python 3.14. I also removed the comments and deprecated suggestion-mode option from .pylintrc which was dropped in Pylint 4.