So based on IRC discussion regarding (2), here is the conclusion:

1. Reading rows instead of columns is better for quick cache warming 
2. Reading rows instead of columns for new metrics that BF has never seen is the case which would be negatively impacted by this change. Since the number of columns per row is small for meta cache, the impact should not be visible at all. So this is a fine change.