Minor Description
the KernelNodeStore defines redundant access methods for the
commitEditor field:
public CommitEditor getEditor()
CommitEditor getCommitEditor()
since there is also a public method to set the CommitEditor i assume
that the public getter was the intended one despite the fact that it
is not used within the code.
if nobody objects i would remove the second method and replace the
single usage of it with the public method.