This enables a new rule that will push down the full `aggregateWindow`
query including the `duplicate` and `window(every: inf)` that recombines
the tables. When the full rule is used, the table is not split into
tables for each window and instead retains itself as a single table. The
start or stop column is renamed to `_time` and `_start` and `_stop` will
be the boundaries of the query.
My original PR influxdata/influxdb#18207 attempted to address the
aggressiveness of our .gitignore matches for known binary names.
Unfortunately, new directories with these names are now being ignored
when they shouldn't (thanks @jsternberg). Upon further investigation, it
looks like the only problematic patterns in the original code were the
plain (non-globbed) patterns like `influxd` and `influxdb`.
Returning the globbed patterns to their original order fixes the problem
while still holding onto the characteristic of being able to see
directories like `cmd/influxd/launcher` via tools that use .gitignore
for exclusion patterns. Below is an overview of how I've verified the
changes.
Regular files matching know binary names should be ignored anywhere in
the tree:
$ echo "hello" > influxd
$ echo "hello" > dbrp/influxd
$ git status
## bb/reorder-exclusion
Directories matching known binary names should not be ignored anywhere
in the tree:
$ mkdir influxd
$ echo "hello" > influxd/hello.txt
$ mkdir dbrp/influxd
$ echo "hello" > dbrp/influxd/hello.txt
$ git status
## bb/reorder-exclusion
?? dbrp/influxd/
?? influxd/
$ git status --untracked-files
## bb/reorder-exclusion
?? dbrp/influxd/hello.txt
?? influxd/hello.txt
New files in existing directories matching known binary names should not
be ignored:
$ echo "hello" > cmd/influx/hello.txt
$ git status
## bb/reorder-exclusion
?? cmd/influx/hello.txt
I've also made changes to already indexed files like
`cmd/influxd/launcher/engine.go` and those changes aren't ignored.
This adds a launcher test for the read window aggregate push down to
verify that it is done when a query is sent with the appropriate
pattern, the output is correct, and that the metric is incremented that
signals the push down happened.
The current .gitignore patterns for accidental commits of binaries like
influxd and influx are too aggressive. It's common for tooling to use
.gitignore to filter out files from listings.
These rules cause the entire cmd/* directory to be excluded before the
exclusion (!*) rules are applied. Per gitignore manfile:
An optional prefix "!" which negates the pattern; any matching file
excluded by a previous pattern will become included again. It is not
possible to re-include a file if a parent directory of that file is
excluded.
The result is that any files/packages inside cmd/* directories cannot be found
by tooling that uses .gitignore for file-filtering.
This adjusts the most permissive of the rules to target project root
only. Files found at other locations in the tree will be ignored
correctly.
Johnny Steenbergen
alexpaxton
Yiqun (Ethan) Zhang
Paul Hummer
Jonathan A. Sternberg
Ariel Salem
Brett Buddin
Bucky Schwarz
Paul Hummer
Faith Chikwekwe
Faith Chikwekwe
Vlasta Hajek
rbose22
Andrew Watkins
Alex Boatwright
Rose Parker
Christopher M. Wolff
Pavel Závora