Remove unnecessary calls for time.Now()

[japinli]

In execute() function, it calls time.Now() to set scrapeDone when there is an error. However, it also set the scrapeDone in Collect(), so it's redundant to set scrapeDone in execute().

[Vonng]

that make sense, thanks~