In flame graphs we saw that not considering the parameter values of a query in the QueryKey#hashCode will lead to many hash collisions. It doesn’t seem to be very useful to include the firstResult/maxResults though IMO, as such queries probably aren’t cached often.

Experiments with SpecJ showed that this improve performance and eliminate collisions in the benchmark scenario.