Hi Max,

On 3/18/2017 5:40 PM, Weijun Wang wrote:
Please take a review on these 2 release notes:

https://bugs.openjdk.java.net/browse/JDK-8177084
8029994: Support "include" and "includedir" in krb5.conf
Only one minor suggestion: replace "say, " with "for example" or "e.g.". "say, " in this context sounds too colloquial for something more formal like a release note.

https://bugs.openjdk.java.net/browse/JDK-8177083
8036779: sun.security.krb5.KdcComm interprets kdc_timeout as msec instead of sec
Again, only minor suggestions:
* Replace "interop" with "interoperability"
* Replace "Customers concerning on interop between..." with "Customers concerned about interoperability between..."

Both were integrated quite some time ago.

Thanks
Max
Both look good to me otherwise.

--Jamil

Reply via email to