Github user marmbrus commented on the pull request:

    https://github.com/apache/spark/pull/332#issuecomment-40646342
  
    It doesn't appear that your code is actually using macros, so I'm not sure 
what the comparison is here.  I'd also caution against reading to much into 
simple micro benchmarks.  Its entirely possible that the JVM is statically 
eliminating the if statement or making other simplifications when it JITs this 
code, so you might just be measuring the time to add an integer.  Also, for 
benchmarks you should [use nanoTime not 
currentTime](http://stackoverflow.com/questions/351565/system-currenttimemillis-vs-system-nanotime).


---
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.
---

Reply via email to