milenkovicm opened a new pull request, #1578: URL: https://github.com/apache/datafusion-ballista/pull/1578
# Which issue does this PR close? Closes #736. # Rationale for this change As mentioned in https://github.com/apache/datafusion-ballista/pull/1388#issuecomment-3980564362 we may get out of ports to establish connection in some cases. Also, sometimes connection establishment may be expensive. Hence we need to provide a way to cache established connections. # What changes are included in this PR? A new pluggable trait `BallistaClientPool` which can support different strategies of client caching # Are there any user-facing changes? changes are backward compatible, as new functionality has to be enabled using `SET ballista.client.pool=true` also, this change supersedes #1388 # TODO: - [ ] configurable pool timeout and purging tread interval - [ ] wire up with executor -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
