RotationOption::SizeMB takes integer argument and may not be fine grained enough

[Zylatis]

I imagine most people don't need to rotate their logs under 1mb in size but potentially certain environments require that, it makes testing easier, and there's no good reason why we shouldn't support it. The question is, do we change the argument of the enum to float or make a separate SizeBytes enum option?