WIP: Murmur: lower priority of ExecEvents to ensure server interactivity under heavy RPC load. #2802

Open
wants to merge 1 commit into
from

Projects

None yet

2 participants

@mkrautz
Member
mkrautz commented Feb 2, 2017

No description provided.

@hacst
Member
hacst commented Feb 2, 2017

Wow. Qt can do that? Sounds like a great change.

@mkrautz
Member
mkrautz commented Feb 2, 2017

Doesn't actually seem to work very well. I tested it by adding a QThread::sleep(1) to MurmurIce.cpp's handling of ExecEvent::execute().

It seems like Qt may be prioritizing at postEvent time?

@mkrautz
Member
mkrautz commented Feb 3, 2017

Actually, maybe the problem is that MurmurIce isn't running in a separate thread? Seemed like it wasn't, but I didn't actually test it yesterday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment