Expose date_histogram->histogram to external crates (#148)

barbarj · rebasedming · commit 45d089575153 · 2026-06-22T12:06:50.000-07:00
As part of supporting all postgres datetimes in ParadeDb, we need to rewrite `date_histogram` requests to regular `histogram` requests. Exposing this helper saves duplicating conversion logic that is not specific to the situation we are converting in. (cherry picked from commit dcbfce2)
diff --git a/src/aggregation/bucket/histogram/date_histogram.rs b/src/aggregation/bucket/histogram/date_histogram.rs
@@ -117,7 +117,7 @@ pub struct DateHistogramAggregationReq {
 }
 
 impl DateHistogramAggregationReq {
-    pub(crate) fn to_histogram_req(&self) -> crate::Result<HistogramAggregation> {
+    pub fn to_histogram_req(&self) -> crate::Result<HistogramAggregation> {
         self.validate()?;
         Ok(HistogramAggregation {
             field: self.field.to_string(),

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ pub struct DateHistogramAggregationReq {`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`impl DateHistogramAggregationReq {`
`120`		`- pub(crate) fn to_histogram_req(&self) -> crate::Result<HistogramAggregation> {`
	`120`	`+ pub fn to_histogram_req(&self) -> crate::Result<HistogramAggregation> {`
`121`	`121`	`self.validate()?;`
`122`	`122`	`Ok(HistogramAggregation {`
`123`	`123`	`field: self.field.to_string(),`