Skip to content

[Backport] index_parallel: support !appendToExisting with no explicit intervals#7113

Merged
jihoonson merged 1 commit intoapache:0.14.0-incubatingfrom
jihoonson:backport-7046
Feb 23, 2019
Merged

[Backport] index_parallel: support !appendToExisting with no explicit intervals#7113
jihoonson merged 1 commit intoapache:0.14.0-incubatingfrom
jihoonson:backport-7046

Conversation

@jihoonson
Copy link
Copy Markdown
Contributor

Backport of #7046 to 0.14.0-incubating.

…pache#7046)

* index_parallel: support !appendToExisting with no explicit intervals

This enables ParallelIndexSupervisorTask to dynamically request locks at runtime
if it is run without explicit intervals in the granularity spec and with
appendToExisting set to false.  Previously, it behaved as if appendToExisting
was set to true, which was undocumented and inconsistent with IndexTask and
Hadoop indexing.

Also, when ParallelIndexSupervisorTask allocates segments in the explicit
interval case, fail if its locks on the interval have been revoked.

Also make a few other additions/clarifications to native ingestion docs.

Fixes apache#6989.

* Review feedback.

PR description on GitHub updated to match.

* Make native batch ingestion partitions start at 0

* Fix to previous commit

* Unit test. Verified to fail without the other commits on this branch.

* Another round of review

* Slightly scarier warning
@jihoonson jihoonson added this to the 0.14.0 milestone Feb 20, 2019
@jihoonson jihoonson merged commit a8ae715 into apache:0.14.0-incubating Feb 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants