Move logic into append_to_builder

[gatesn]

Currently append_to_builder for canonical arrays looks something like:

    fn _append_to_builder(&self, builder: &mut dyn ArrayBuilder) -> VortexResult<()> {
        builder.extend_from_array(self)
    }

Where the builder presumably performs some array downcast.

Instead, the encodings should unpack themselves into the builders.