Merge lp:~akiban-technologies/akiban-persistit/releases_notes_3.1.2_2 into lp:akiban-persistit

Proposed by Peter Beaman
Status: Merged
Approved by: Nathan Williams
Approved revision: 333
Merged at revision: 334
Proposed branch: lp:~akiban-technologies/akiban-persistit/releases_notes_3.1.2_2
Merge into: lp:akiban-persistit
Diff against target: 56 lines (+4/-8)
3 files modified
doc/ReleaseNotes.rst (+3/-5)
src/main/java/com/persistit/AlertMonitor.java (+0/-2)
src/main/java/com/persistit/SharedResource.java (+1/-1)
To merge this branch: bzr merge lp:~akiban-technologies/akiban-persistit/releases_notes_3.1.2_2
Reviewer Review Type Date Requested Status
Nathan Williams Approve
Review via email: mp+114864@code.launchpad.net

Description of the change

Fix a couple of typos in the release notes and a couple of Javadoc warnings in the code

LBJ: NO_BUILD

To post a comment you must log in.
Revision history for this message
Nathan Williams (nwilliams) wrote :

Looks good.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'doc/ReleaseNotes.rst'
2--- doc/ReleaseNotes.rst 2012-07-12 21:06:00 +0000
3+++ doc/ReleaseNotes.rst 2012-07-13 13:39:21 +0000
4@@ -88,7 +88,7 @@
5
6 https://bugs.launchpad.net/akiban-persistit/+bug/1017957
7
8-This bug preexisted but was unknown to the 3.1.1 release. An extremely rare combination of events caused memory structures caused an incorrect key-pointer pair to be inserted while deleting records in a key range. The bug was detected in an 8-hour stress test run.
9+This bug preexisted but was unknown to the 3.1.1 release. An extremely rare combination of events corrupted memory structures causing an incorrect key-pointer pair to be inserted while deleting records in a key range. The bug was detected in an 8-hour stress test run.
10
11
12 Slow Recovery Due to Temporary Tree IT Records
13@@ -96,15 +96,13 @@
14
15 https://bugs.launchpad.net/akiban-persistit/+bug/1018526
16
17-This bug preexisted but was unknown to the 3.1.1 release.
18-
19-Every Tree created in a temporary volume was being assigned a tree handle recorded permanently in the journal. In one case the result was a journal containing millions of IT (Identify Tree) records, and these caused normal recovery to take a very long time. The fix keeps temporary trees out of the journal and removes IT records which may have been added previously.
20+This bug preexisted but was unknown to the 3.1.1 release. Every Tree created in a temporary volume was being assigned a tree handle recorded permanently in the journal. In one case the result was a journal containing millions of IT (Identify Tree) records, and these caused normal recovery to take a very long time. The fix keeps temporary trees out of the journal and removes IT records which may have been added previously.
21
22
23 Asserts Added to Check for Correct Exchange Thread Behavior
24 -----------------------------------------------------------
25
26-A bug in the Akiban Server code caused a an Exchange to be used concurrently by two Threads, causing serious and seemingly unrelated failures in Persistit including instances of IllegalMonitorException and IllegalStateException. To guard against future occurrences, asserts were added to catch such concurrent use by multiple threads. Applications should be tested with asserts enabled to verify correct thread usage.
27+A bug in the Akiban Server code caused an Exchange to be used concurrently by two Threads, causing serious and seemingly unrelated failures in Persistit including instances of IllegalMonitorException and IllegalStateException. To guard against future occurrences, asserts were added to catch such concurrent use by multiple threads. Applications should be tested with asserts enabled to verify correct thread usage.
28
29
30 Known Issues
31
32=== modified file 'src/main/java/com/persistit/AlertMonitor.java'
33--- src/main/java/com/persistit/AlertMonitor.java 2012-06-22 19:35:13 +0000
34+++ src/main/java/com/persistit/AlertMonitor.java 2012-07-13 13:39:21 +0000
35@@ -537,8 +537,6 @@
36 * @param category
37 * A String describing the nature of the event. A separate
38 * event-history is maintained for each unique category.
39- * @param level
40- * Indicates whether this event is a warning or an error.
41 */
42 public synchronized final void post(Event event, final String category) {
43 History history = _historyMap.get(category);
44
45=== modified file 'src/main/java/com/persistit/SharedResource.java'
46--- src/main/java/com/persistit/SharedResource.java 2012-06-29 20:54:00 +0000
47+++ src/main/java/com/persistit/SharedResource.java 2012-07-13 13:39:21 +0000
48@@ -48,7 +48,7 @@
49 * Default maximum time to wait for access to this resource. Methods throw
50 * an InUseException when this time is exceeded.
51 */
52- final static long DEFAULT_MAX_WAIT_TIME = 60000L;
53+ public final static long DEFAULT_MAX_WAIT_TIME = 60000L;
54
55 /**
56 * Mask for count of Threads holding a reader or writer claim (0-32767)

Subscribers

People subscribed via source and target branches

to all changes: