Skip to content

Improve jackson benchmark in java#303

Merged
xerial merged 3 commits intov07-developfrom
v07-improve-jackson-benchmark-in-java
Dec 25, 2015
Merged

Improve jackson benchmark in java#303
xerial merged 3 commits intov07-developfrom
v07-improve-jackson-benchmark-in-java

Conversation

@komamitsu
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The order of running programs also affects the performance results in JVM. Simply running System.gc() as an warmup is not sufficient.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The benchmark has 4 times warmup runs for each one and it avoids the affect of running order, I think. BTW, how does your benchmark avoid the order of running programs?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For example, if you have two programs P1, P2, it runs the program P1, P2, P1, P2,... to reduce the order effect.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@komamitsu
Copy link
Copy Markdown
Member Author

@xerial Can you review it again? Thanks

@xerial
Copy link
Copy Markdown
Member

xerial commented Dec 25, 2015

ok. It looks good.

xerial added a commit that referenced this pull request Dec 25, 2015
@xerial xerial merged commit d154717 into v07-develop Dec 25, 2015
@komamitsu komamitsu deleted the v07-improve-jackson-benchmark-in-java branch December 25, 2015 04:31
@komamitsu komamitsu restored the v07-improve-jackson-benchmark-in-java branch December 25, 2015 07:00
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