Thanks Dag for reviewing in such details. I feel I shall help more on
DERBY-4741 since you help me so much.
Some further legibility improvement suggestions:
> Summary of Change
> Derby will by default dump as much thread dump and diagnostic
> information as possible on system crash or session error.
> i.e. users will find thread dump information with error severity
> higher than session error in derby.log.
On experiencing a system crash or session error, Derby will print as
much diagnostic information as possible to the Derby error log,
including stack traces of all threads.
>>I change to reflect the suggestion.
> It is also possible to get thread dump and diagnostic information of
> errors with lower severity by adjusting a property.
This one is good, maybe change "of" to "for",
> Symptoms Seen by Applications Affected by Change
> If errors have error severity level greater or equal than value
> derby.stream.error.extendedDiagSeverityLevel is set, thread dump
> information and extened diagnostic information will be handle by
> Derby. Users can find thread dump information in derby.log. For ibm
> JVM users, a javacore file will be created.
I still feel this information belongs in the first section, and that
this section should be N/A. I agree this is useful
"handle" -> "printed"
"in derby.log" -> "in the Derby error log"
ibm -> IBM
>>I see your point. I still feel it does not hurt to be here for users who only view this section for release note.
> Incompatibilities with Previous Release
> Rationale for Change
> The intention of the change to get more detail information relate to
"relate" -> "related"
> the status of Derby when user hit system crash or server session
"the status of Derby when important errors happen, "important" by
default meaning errors with session severity and higher. This will
help users, DBAs and support personnel to determine the exact cause of
the error condition."
>>I change the release note to reflect to the suggestion.
> Application Changes Required
> Users do not need to change anything for this change. If more thread
> dump information is required for error that have severity level less
> than session error, derby.stream.error.extendedDiagSeverityLevel can
"the property derby.stream.error.extendedDiagSeverityLevel"
> be set in the file "derby.properties" or as part of the JVM
In stead of specifying "derby.properties" or JVM argument here,
wouldn't it be better to just state this is a Derby system property
and give a link to the documentation? System wide properties can also
be set programmatically. You could also say, here as an example, "for
example, in derby.properties" but not try to enumerate all ways it can
>>Thank you so much for the link. I was trying find the link and add it to it. The users will see the link in this releaseNote.html.
> For example, if there is a deadlock issue and users would
> like to see thread dump information while the deadlock is happening,
> users can then set
> derby.stream.error.extendedDiagSeverityLevel=30000 in the file
> "derby.properties" or as part of JVM argument,
> e.g. -Dderby.stream.error.extendedDiagSeverityLevel=30000.
See above comment, drop how to to set it here altogether, just show
the value required.
> The thread dump information is found in the Derby error log,
"The thread dump information" ->
"The diagnostics and the thread dump information"
> typically this is the file "derby.log".
> means turn on TRANSACTION_SEVERITY type of error that the extended
> thread dump information for errors of transaction severity or
"derby.stream.error.extendedDiagSeverityLevel=30000 means that extra
diagnostics and thread dump will be activated for errors with
severity equal to og higher than transaction severity (30000)."
>>The change is reflected to the suggestion.
> Please consult the Derby documentation for explanation of error
> severity and for other possible settings of