This seems a bit redundant with `getPoolsInfo`. What if you remove getPoolsInfo and just call this getPools with a docstring that explains what it returns. Then if people just need the list of pools they can call `val pools = sc.getPools.values`