[ATLAS-3134] Change Date.getTime() to System.currentTimeMillis() to improve performance

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.1.0
• Flags: Patch
• Assignee: Unassigned
• Reporter: bd2019us
• Created: 12/Apr/19 03:33
• Updated: 29/Jun/19 00:03
• Resolved: 24/Jun/19 06:34

Description

new Date() is a thin wrapper of method System.currentTimeMillis(). If it is intensively invoked in the program, the performance will be greatly damaged. 
According to my local testing, when these two methods are invoked 5,000,000 times at the same environment, System.currentTimeMillis() can achieve a speedup to 5 times (435ms vs 2073ms).
Therefore, if only getTime() is used for Date object, the light method System.currentTimeMillis() is highly recommended, which can also avoid creating the temporary Date object.
 

Issue Links

• links to: GitHub Pull Request #40