flux-framework / dyad

DYAD: DYnamic and Asynchronous Data Streamliner
GNU Lesser General Public License v3.0
7 stars 5 forks source link

Add caching of UCX endpoints to prevent excessive creation #33

Closed ilumsden closed 2 months ago

ilumsden commented 1 year ago

Yet another suggestion from @hariharan-devarajan

We should add code to temporarily cache recently opened UCX endpoints. By doing so, we can minimize how much time we spend opening network connections.

Before starting this, we need to decide a caching strategy. I'd personally suggest LRU as a starting point.

hariharan-devarajan commented 1 year ago

Yes LRU seems a good initial approach. We can look at the literature of networks and see if there are any other better (but simple approaches like LRU) out there. I am tempted to support multiple policies if we see an inherent trade-off between different policies.

ilumsden commented 2 months ago

Initial caching is added