kstyrc / embedded-redis

Redis embedded server for Java integration testing
840 stars 368 forks source link

Further documentation to tell user not configure logfile #64

Open loyalsound opened 8 years ago

loyalsound commented 8 years ago

Today, I'm get stucks when starting application (Spring Boot) when trying to configure logfile option (I know this project not intent to be run in production system. But I think that it realiable enough to act as a cache store).

When log redirected to file, application will wait forever. So please document that to make easy of use.

Thanks!

tacticiankerala commented 7 years ago

I also bumped into the same problem. I configured logfile and the application was stuck.

@loyalsound , could you help me out why it is not advised to use this in production?

tacticiankerala commented 7 years ago

I guess the problem is with awaitRedisServerReady(); in the AbstractRedisInstance.java file. When the output is redirected to a logfile, the library is still trying to read the redis start pattern from the input stream which is empty and it waits forever.

tacticiankerala commented 7 years ago

I have a naive fix for this issue. Please find it here. Based on the feedback I can work on it further.