MINOR: code cleanup in InternalTopicManager

[mjsax]

• Avoid double-brace initialization
• Fix timeout error handling, by pass in correct list of topic names
• Remove unnecessary code

[Pankraz76]

interesting syntax. Its like an array init, within the declaration, right?

Does is make an performance difference, or is it just error prone, thus some kind of community convent to avoid?

[mjsax]

Its like an array init, within the declaration, right?

Yes. Similar.

Does is make an performance difference, or is it just error prone, thus some kind of community convent to avoid?

It's not considered best practice. Overlooked during code-review. From Copilot:

Using double-brace initialization creates an anonymous inner class and can lead to memory leaks. Consider using a standard HashSet construction with addAll() calls or collect the items into a set using streams.