Add some comments on loggin