-
1. Re: AIO error
clebert.suconic May 29, 2008 12:28 PM (in response to timfox)The client is starting the journal? It shouldn't be doing it.
I would need to look at what changed? Maybe someone already know it. -
2. Re: AIO error
timfox May 29, 2008 12:31 PM (in response to timfox)What do you mean "the client is starting the journal"?
-
3. Re: AIO error
clebert.suconic May 29, 2008 12:51 PM (in response to timfox)What do you mean "the client is starting the journal"?
NM.. ignore me... i could replicate the error.
I thought the error was happening on the client side. My bad interpretation. -
4. Re: AIO error
clebert.suconic May 30, 2008 1:30 AM (in response to timfox)As the close and open are both asynchronous, at high rates we could have 5 or 6 files opened what would maximize the max AIO on the OS.
I want to add a retry block on open, but since that will require some minor changes on the native layer I will leave it for Beta. My concern is this excpetion is thrown by a constructor and I'm concerned If we keep retrying to open the file, hitting the constructor and ignoring the memory created. I don't know what are the consequences and I don't have enough time to test this before alpha. A simple and easy change.. but requires testing. (It is almost done on my workspace BTW.. but I won't commit it now)
The native layer is stable for a while already and I would like to keep as is for Alpha now.
I have added these two tasks to revisit this on Beta:
http://jira.jboss.org/jira/browse/JBMESSAGING-1350 - Revisit Error codes and messages on exceptions thrown by the native layer
http://jira.jboss.org/jira/browse/JBMESSAGING-1349 - Retry blocks on asynchronous open
Maybe I will make one tiny change on the native layer before the release that wouldn't cause any issues though... just to change that error message. (Can't initialize aio). It is an optional task. -
5. Re: AIO error
clebert.suconic May 30, 2008 1:31 AM (in response to timfox)I forgot to say that for the release... making MaxAIO=5000 on jbm-configuration is good enough for Alpha what fixed this error message already.