Folks,

As I see important performance improvements were merged to the *master *branch.
One of them is "striped pool" which can be used instead of current system
pool. I think it's configuration is not very good at the moment:

int IgniteConfiguration.setStripedPoolSize();

Problems:
1) It has non-zero default meaning that it will be used by default. It is
ok, but if user previously configured system pool size with
"IgniteConfiguration.setSystemPoolSize()", then his config will be ignored
silently. It looks wrong to me to have two configurable sizes for a single
pool.
2) What if decide to add striped pool optimization for other pools in
future, e.g. for public pool? How are we going to name configuration
properties?

I think we should do the following:
1) Remove *"int stripedPoolSize"* property.
2) Add *"boolean systemThreadPoolStriped"* property, set it to *true *by
default.
3) Rely on existing *"int systemThreadPoolSize"* property to get number of
threads.

Thoughts?

Vladimir.

Reply via email to