should be the correct way to gorupby