stop using legacy indexes

in this case `relationship_type` is the uppercase neo relationship name,
and we'll return the type_id which (effectively) is the relationship class name - is that correct?
Is the word *type* a bit overloaded here - just wondering if there's some other terminology we can use?