On 9/9/11 2:40 PM, Myrna van Lunteren wrote:
On Fri, Sep 9, 2011 at 8:53 AM, Dag H. Wanvik<[email protected]>  wrote:
Hi,

thanks for assembling this, Myrna.

Myrna van Lunteren<[email protected]>  writes:

  DERBY-5331 Incorrect use of CharsetEncoder in DDMWriter
  DERBY-5354 Remove unnecessary dita.regex build target
  DERBY-5318 Use assertDirectoryDeleted in ReplicationRun and remove dead code
  DERBY-5314 Enable i18n tests in non-English locales
  DERBY-5291 testDerby4137_TransactionTimeoutSpecifiedNotExceeded
  DERBY-5262 Running JUnit tests with Java 1.4.2 fails if the package private
  DERBY-5253 Engine code references
  DERBY-5252 make GrantRevokeTest pass in non-English locale
  DERBY-5251 make ErrorCodeTest pass in non-English locale
  DERBY-5247 Warnings regarding XPath displayed when generating JavaDoc
  DERBY-5211 Make SysinfoCPCheckTest pass in non-English locale
  DERBY-2625 SEVERE error involving column-width property
  DERBY-2623 SEVERE error involving column-number property
  DERBY-3337 convert jdbcapi/derbyStress.java to JUnit
  DERBY-5084 convert ijConnName.sql to a ScriptTest junit test
  DERBY-1903 Convert largedata/LobLimits.java to junit
These are not user visible changes and could be left out, I think.

Rest looks good to me!

Thanks,
Dag

Thanks for the review (Rick too)!

I did a little experiment, to see if I could tweak my filter, and I
succeeded in getting most - but not all - issues you mentioned by
removing the 'Test' component. I didn't think I should manually adjust
the release notes.
That sounds like the right approach. Hand-editing the release notes would be a brittle solution.
However, I then checked on what Rick has done for e.g. 10.8.1 and test
issues, which end-users wouldn't see, were included in the release
notes for that release also, look e.g. for DERBY-5174 in those.

I remember - maybe incorrectly - that Apache instructs us to include a
record of *all* changes that went into a release, I think that's why
we're including test and build changes (as well as changes to source
that hasn't been in any official release). I think this is related to
the fact that our apache release also includes a source distribution.
I believe that a similar discussion came up in the past about whether we should exclude documentation changes from our release notes. I imagine there are people out there who are only interested in behavioral changes to the product. The extra detail in our release notes won't be useful to them.

I don't get the sense that we're swamping signal with noise. However, that's just my opinion.
So I think those bugs will stay in the release notes...

Rick, as the most recent release manager, if I'm going off track with
my reasoning, let me know...
If people feel strongly that the release notes are too verbose, then we should discuss how to flag noise issues in JIRA. That way we can programmatically exclude them from the filters. It's late in the day to do this for 10.8.2 but we could consider this change for 10.9.

Thanks,
-Rick
Myrna


Reply via email to