should this be a logger call instead of printStackTrace?