previously we incorrectly identified matching buckets by comparing string pointers. this worked in the unit test but not in real applications since the strings result from date parsing inside the facet collector, and are therefore different pointers