Github user yjshen commented on the pull request:
https://github.com/apache/spark/pull/7870#issuecomment-127494555
I've updated the
[benchmark](https://gist.github.com/yjshen/2c48cfe6201a97e6e1f5) again, really
sorry for so many mistakes.
In order to avoid cache in Calendar, in each method, I call the original
code twice with different day.
```
DateTimeUtilsBenchmark.getDayOfMonth avgt 10 53.252 ± 8.566
ns/op
DateTimeUtilsBenchmark.getDayOfMonthOld avgt 10 32.052 ± 3.525
ns/op
DateTimeUtilsBenchmark.getDayOfMonthJava avgt 10 200.894 ± 18.744
ns/op
DateTimeUtilsBenchmark.getMonth avgt 10 53.408 ± 7.292
ns/op
DateTimeUtilsBenchmark.getMonthOld avgt 10 31.612 ± 4.579
ns/op
DateTimeUtilsBenchmark.getMonthJava avgt 10 211.795 ± 21.510
ns/op
```
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
