That name doesn't tell at all what it should do. Maybe something like get_contents_from_range or inflate_range.

Should be renamed to something like list_content_to_index.

Set[bytes]

That loop runs either a single time or forever. You should rewrite the loop to:

while start:
    # do the thing
    start = result['next']

representing

Any reason to use a 2-tuple, rather than just taking start and end as parameters?

set()

Same issue as ContentIndexer._new's loop.

ContentIndexer._new and MimetypeRangeIndexer.range seem to do very similar jobs. Shouldn't they be merged somehow?

wtf, that code is completely broken.

More indent on this line?

\m/

I kept the original args of the base indexer.

Yes, that sounds appropriate (given how i messed up early on ;) but i'd like correctness first.

Well, now that the orchestrator is out of the way, might be the conditional is no longer relevant (i think so).

huh, right!

In the end, no. flake8 complains otherwise:

./swh/indexer/tests/test_mimetype.py:82:17: E125 continuation line with same indent as next logical line

Hum, that should not be here, removing.

Thanks. I settled on indexed_contents_in_range.

Ah and also i did not realize i could simply change the parameters ;)

Yes, it returns the result (that's what i did below).
I did not touch that part except for adding a docstring.
It's out of scope, i'll change in another diff.
I'll probably rename the task WithResultTask or something.

Build has FAILED

That's why. Rebase error, that should be a list.
Fixing this.

yes