Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ISPN-7362 Use no-bundler in the default UDP configuration #4763

Merged
merged 1 commit into from Jan 23, 2017

Conversation

danberindei
Copy link
Member

@danberindei danberindei commented Jan 14, 2017

@danberindei
Copy link
Member Author

run performance tests please

@ghost
Copy link

ghost commented Jan 14, 2017

Performance tests run successfully. Link to the results here.

@danberindei danberindei force-pushed the ISPN-no-bundler branch 2 times, most recently from 89fc620 to 3414b80 Compare January 16, 2017 15:02
@Holmistr
Copy link

run performance tests please

@Holmistr
Copy link

Holmistr commented Jan 17, 2017

@danberindei Results were successfully generated, just ignore the C/S part: Link to the report here Just to clarify - these are the results from your previous request, currently there is one more test in progress triggered by my comment above. Estimated finish in 1 hour.

@ghost
Copy link

ghost commented Jan 17, 2017

Performance tests run successfully. Link to the results here.

@danberindei danberindei changed the title [PREVIEW] Use no-bundler in the default UDP configuration ISPN-7362 Use no-bundler in the default UDP configuration Jan 17, 2017
@danberindei
Copy link
Member Author

Results with no-bundler look good. Keeping transfer-queue and re-adding DONT_BUNDLE didn't improve anything.

@pruivo
Copy link
Member

pruivo commented Jan 17, 2017

@danberindei which results are you looking at? in the latest results, there is no improvement (actually, it is worse)

@danberindei
Copy link
Member Author

@pruivo I reused the PR to test the transfer-queue bundler with the DONT_BUNDLE flag, and that's what the latest results show. The only run with no-bundler is http://perfrepo.mw.lab.eng.bos.redhat.com/reports/tableComparisonReport/4716

@danberindei danberindei force-pushed the ISPN-no-bundler branch 4 times, most recently from b0d2ff6 to 2fd2881 Compare January 20, 2017 15:18
@tristantarrant tristantarrant merged commit 7a8ad3c into infinispan:master Jan 23, 2017
@tristantarrant
Copy link
Member

Pushed, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants