Jon let the Umbers and Karstarks keep their lands - was that a wise decision?

Or was Sansa right - treason needs to be punished, and loyalty rewarded?