It would probably be best to just make this a def, so FailedStageTable#columns can just append to the list. isFairScheduler should probably also be a def, as checking the volatile variable is almost certainly more expensive than a method call to check a boolean.