Java View.beginIndex doesn't support multiple view indexing about cbforest HOT 4 CLOSED

This blocks to fix https://github.com/couchbase/couchbase-lite-java-core/issues/681

from cbforest.

The intent is that you can create multiple c4Indexers at the same time, and then iterate through the docs once, feeding each indexer its keys and values.

Looking at the API, what's missing is a version of c4indexer_enumerateDocuments that takes multiple indexers as input, computes the earliest sequence that needs to be indexed, and starts from there. Should be easy to do.

from cbforest.

@snej I think the enumerateDocuments logic that you described above has already been implemented as C4Indexer extended from MapReduceIndexer. The only missing piece is that multiple views if available should be given to c4indexer_begin() instead of only just 1 from the current view (See beginIndex code). Probably we also need to extract an Indexer class from the View class in order to do that.

from cbforest.

Just uploaded PR #48 to support this.

from cbforest.