Added filtered time series retrieval API. - #1169
Conversation
Introduced `TimeSeriesRequestParameters` and 'FilteredTimeSeriesParameters' to clean up TimeSeriesDaoImpl. Added RSQL query capability
|
Replacement PR b/c #1158 got closed when I force-pushed a new branch that didn't have the weird self-merge in its history |
|
@MikeNeilson Last we talked you suggested merging this and letting people give feedback |
|
I was hoping some feedback would be provided on the PR, but merge and let be used is likely the more practical way at this point. |
|
@MikeNeilson Should I remove the explicit min-value, max-value and filter-nulls parameters from the controller? With the filter-expression-language implemented those features can be achieved in an RSQL query. Removing them might force people to figure out how to build their query? |
|
That seems rational to me, doesn't really make sense to have both and need to support two different paths. |
|
@MikeNeilson This is ready now |
Introduced `TimeSeriesRequestParameters` and 'FilteredTimeSeriesParameters' to clean up TimeSeriesDaoImpl. Added RSQL query capability
2 participants
Introduced
TimeSeriesRequestParametersand 'FilteredTimeSeriesParameters' to clean up TimeSeriesDaoImpl. Added RSQL query capability