Specifically it should be considered whether Vec fields such as error_bounds and generated_columns on ModelTableMetadata can use HashMap instead to avoid issues with indices being error prone and harder to understand. When saving tag values in structs we also use a vector of strings which could also be changed to a HashMap.
Generally, whenever we save per-column data, it can be considered whether it would be better to use a HashMap instead of a Vec. A discussion on this was started here: #293 (comment).
Specifically it should be considered whether
Vecfields such aserror_boundsandgenerated_columnsonModelTableMetadatacan useHashMapinstead to avoid issues with indices being error prone and harder to understand. When saving tag values in structs we also use a vector of strings which could also be changed to aHashMap.Generally, whenever we save per-column data, it can be considered whether it would be better to use a
HashMapinstead of aVec. A discussion on this was started here: #293 (comment).