Skip to content

Commit

Permalink
Loosen checks in REST test for PIT slicing (elastic#75261)
Browse files Browse the repository at this point in the history
Since the index may have more than one shard, we can't always predict how many
documents will fall in each slice. This PR simply removes the checks, since we
already test the slicing logic extensively in an integration test. The REST test
is now just a sanity check for the API.

Fixes elastic#75212.
  • Loading branch information
jtibshirani authored and masseyke committed Jul 16, 2021
1 parent 0c2f728 commit 10ed0cb
Showing 1 changed file with 2 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -136,8 +136,8 @@ setup:
---
"point-in-time with slicing":
- skip:
version: "all"
reason: "https://github.com/elastic/elasticsearch/issues/75212"
version: " - 7.14.99"
reason: "support for slicing was added in 7.15"
- do:
open_point_in_time:
index: test
Expand All @@ -158,9 +158,6 @@ setup:
pit:
id: "$point_in_time_id"

- match: {hits.total.value: 2 }
- length: {hits.hits: 1 }

- do:
close_point_in_time:
body:
Expand Down

0 comments on commit 10ed0cb

Please sign in to comment.