Revert #1117: change default recycler pool back to `threadLocalPool()` for 2.17.1
cowtowncoder opened this issue · 1 comments
cowtowncoder commented
Due to various reported performance issues (even if not all fully evaluated or confirmed), it seems that change #1117 may be too aggressive for 2.x release; or at least 2.17.x.
Since users may reconfigure implementation anyway (and are encouraged to do so explicitly), I think we should revert default back for 2.17.1 and keep it there for 2.17.x.
Based on research (see #1260) plan is to change default impl concurrentDequePool
for 2.18(.0), via #1266.
cowtowncoder commented
Supporting this change:
- Trino issue: trinodb/trino#21356