[jira] [Commented] (ROCKETMQ-269) don’t need cal.setTimeInMillis(System.currentTimeMillis()) after Calendar.getInstance()

[ASF GitHub Bot (JIRA)]

    [ 
https://issues.apache.org/jira/browse/ROCKETMQ-269?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16128529#comment-16128529
 ] 

ASF GitHub Bot commented on ROCKETMQ-269:
-----------------------------------------

Github user coveralls commented on the issue:

    https://github.com/apache/incubator-rocketmq/pull/148
  
    
    [![Coverage 
Status](https://coveralls.io/builds/12855105/badge)](https://coveralls.io/builds/12855105)
    
    Coverage decreased (-0.07%) to 38.713% when pulling 
**776ddb918f15e8c17809f651ca83c2172c534bb7 on mark800:iss0816** into 
**421a22c73bfb5d1c4748f7d23b0a2a4fd5586192 on apache:master**.



> don’t need cal.setTimeInMillis(System.currentTimeMillis()) after 
> Calendar.getInstance()
> ---------------------------------------------------------------------------------------
>
>                 Key: ROCKETMQ-269
>                 URL: https://issues.apache.org/jira/browse/ROCKETMQ-269
>             Project: Apache RocketMQ
>          Issue Type: Improvement
>          Components: rocketmq-commons
>            Reporter: Mark Yang
>            Assignee: Jixiang Jin
>
> don’t need cal.setTimeInMillis(System.currentTimeMillis()) after 
> Calendar.getInstance()
> In JDK’s source code, innermost constructor for Calendar.getInstance() is:
> public GregorianCalendar(TimeZone zone, Locale aLocale) {    
>  super(zone, aLocale);     gdate = (BaseCalendar.Date) 
> gcal.newCalendarDate(zone);     
>  setTimeInMillis(System.currentTimeMillis()); 
> }



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)