Skip to content

Conversation

@omalley
Copy link
Contributor

@omalley omalley commented Jul 11, 2018

I also refactored all of the old benchmarks to reduce the common code. I also split it into three modules so that I could
separate the common code, the code that depends on hive, and the code that depends on spark. Avoiding building an uber
jar that has both hive and spark made life much easier.

I also refactored all of the old benchmarks to reduce the common code. I also split it into three modules so that I could
separate the common code, the code that depends on hive, and the code that depends on spark. Avoiding building an uber
jar that has both hive and spark made life much easier.
@omalley
Copy link
Contributor Author

omalley commented Jul 13, 2018

@dongjoon-hyun if you have feedback on these benchmarks, that would be great.

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM.

For #21582, I've got the following result from DecimalBench.

# Run complete. Total time: 00:05:40

Benchmark               (version)  Mode  Cnt         Score       Error  Units
DecimalBench.read        ORIGINAL  avgt    5  10375991.566 ± 99751.161  us/op
DecimalBench.read   USE_DECIMAL64  avgt    5   7368917.561 ± 39257.332  us/op
DecimalBench.write       ORIGINAL  avgt    5    211008.673 ± 30220.403  us/op
DecimalBench.write  USE_DECIMAL64  avgt    5     33881.693 ±   216.601  us/op

@asfgit asfgit closed this in a621181 Jul 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants