Mateusz "Serafin" Gajewski
Mateusz "Serafin" Gajewski
@dain: ``` List runningQueries = queryTracker.getAllQueries().stream() .filter(query -> query.getState() == RUNNING) .collect(toImmutableList()); ``` this copies running queries to a new list
@dain agreed but then on every usage site I'm doing ::stream so what's the point?
@dain yeah as an internal API it's easy to change it back when usage patter changes
I'm fine with a drop but let's hear @martint @mosabua thoughts on this one
@findepi should we just lower the stream size or allocate couple of linked buffers instead?
@martint can you take a look?
CI hit: https://github.com/trinodb/trino/actions/runs/19423801211/job/55595664284?pr=27333
@YuriyGavrilov I don't have enough context here to decide whether fix is correct or not so I defer decision to @martint or @findepi