Redundancy is okay and expected. There's lots of examples already, e.g. dwarf or elf fighters. But other features of the race should shine. When I make a mountain dwarf fighter, the proficiencies are subsumed, but the darkvision and +2 Str & Con reinforce the archetype.
I would've preferred races get a fixed +1 based on racial archetype, and an overlaying +1/+1 placed as desired.
Making +2/+1 based on class, when class already has a default array, seems kind of pointless. Might as well just omit it entirely.
