[SERVER-57629] change $integral&$derivative argument from 'outputUnit' to 'unit' Created: 10/Jun/21 Updated: 08/Jul/21 Resolved: 25/Jun/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 5.0.0-rc5 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | David Percy | Assignee: | Jacob Evans |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||
| Backwards Compatibility: | Minor Change | ||||||||||||||||||||||||
| Operating System: | ALL | ||||||||||||||||||||||||
| Backport Requested: |
v5.0
|
||||||||||||||||||||||||
| Sprint: | Query Optimization 2021-07-26 | ||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||
| Description |
|
In the $integral and the $derivative window functions, 'outputUnit' specifies how to express time deltas as a number. We originally called it 'outputUnit' to avoid a name collision with the 'unit' used for the window bounds. This name is confusing though, because often the measurements implicitly have a unit involving time. For example, suppose you have numbers representing meters/second, and times represented as a datetime. The result will have units of meters, so writing outputUnit: 'second' is counterintuitive. Now that the 'window' arguments are separate, we could go back to calling this argument just 'unit':
|
| Comments |
| Comment by Githook User [ 25/Jun/21 ] |
|
Author: {'name': 'Jacob Evans', 'email': 'jacob.evans@10gen.com'}Message: |
| Comment by Githook User [ 25/Jun/21 ] |
|
Author: {'name': 'Jacob Evans', 'email': 'jacob.evans@10gen.com'}Message: |
| Comment by Joe Kanaan [ 17/Jun/21 ] |
|
Potentially need to update the fuzzer and a benchmark |