-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
iss993: added DefaultProperties annotation [ready for review] #1259
Conversation
PR in progress... |
Is this ready for review? |
Not yet, in progress. Will let you know when it's ready.
|
…n: groupKey, threadPoolKey, commandProperties, threadPoolProperties
@mattrjacobs it's ready for review. BTW is it possible to use java 8 ? I'm still using java 6 for javanica. |
If I recall, 1.x is maintaining JVM 6. If not I'd ask to wait for 2.0 to change that. |
It's ok. Will refactor it latter once 2.0 released. If PR is ok you can
|
Yes, let's hold off on Java8 until 2.0 |
LGTM. Thanks @dmgcodevil ! I'll close #993. I'll get master in shape for a 1.5.4 release as well |
fix for #993