In your constructor you don't have to check for a bless to be successful (can't think of any scenario in which that would ever fail), but you should check if the mkdir is successful.

But otherwise it looks ok (I don't like shifts though, I prefer the my($self, $sql)=@_; style but that is mainly a matter of taste).