Bug 1038585

Summary: Documentation of transactions
Product: [JBoss] JBoss Data Grid 6 Reporter: Radim Vansa <rvansa>
Component: DocumentationAssignee: Misha H. Ali <mhusnain>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 6.2.0CC: jdg-bugs, rvansa
Target Milestone: GA   
Target Release: 6.2.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-01-16 00:01:51 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Administration & configuration guide
none
Developer guide none

Description Radim Vansa 2013-12-05 12:23:54 UTC
Created attachment 833116 [details]
Administration & configuration guide

I have reworked the documentation of transactions in both Admin&config guide and developer guide.
Please see the attached documents.

Comment 1 Radim Vansa 2013-12-05 12:24:38 UTC
Created attachment 833117 [details]
Developer guide

Comment 5 Radim Vansa 2013-12-05 14:54:08 UTC
Wow, you're fast! Few details yet

22.1.4. Write Skew Checks

"Use the REPEATABLE_READ isolation level to enable write skew check" -> this is formulated incorrectly. REPEATABLE_READ is needed, but it alone does not enable the write skew check.

"With optimistic transactions, write skew checks are required for (atomic) conditional operations." - please, put this into Important box.

Procedure 22.1., step 1) - please fix the indentation of ellipsis

Procedure 22.2. - could you enable the code highlighting?
step 2) "avilable" -> "available"

Comment 6 Misha H. Ali 2013-12-05 15:06:36 UTC
(In reply to Radim Vansa from comment #5)
> Wow, you're fast! Few details yet
> 
> 22.1.4. Write Skew Checks
> 
> "Use the REPEATABLE_READ isolation level to enable write skew check" -> this
> is formulated incorrectly. REPEATABLE_READ is needed, but it alone does not
> enable the write skew check.

OK, rephrased a bit. 

> "With optimistic transactions, write skew checks are required for (atomic)
> conditional operations." - please, put this into Important box.
Done

> Procedure 22.1., step 1) - please fix the indentation of ellipsis
Fixed.

> Procedure 22.2. - could you enable the code highlighting?
Sorry, my bad. This should be OK now.

> step 2) "avilable" -> "available"
Fixed!

Comment 7 Radim Vansa 2013-12-06 10:16:48 UTC
(In reply to Misha H. Ali from comment #6)
> (In reply to Radim Vansa from comment #5)
> > Wow, you're fast! Few details yet
> > 
> > 22.1.4. Write Skew Checks
> > 
> > "Use the REPEATABLE_READ isolation level to enable write skew check" -> this
> > is formulated incorrectly. REPEATABLE_READ is needed, but it alone does not
> > enable the write skew check.
> 
> OK, rephrased a bit. 
> 

I might be nitpicking, but using "to enable" sounds like you make WSC working by using the isolation level and then the entry versioning. This paragraph is not a procedure for enabling WSC, it lists necessary conditions. None of those will enable WSC.

Comment 9 Misha H. Ali 2014-01-16 00:01:51 UTC
The fix for this bug is now generally released and available here:

https://access.redhat.com/site/documentation/en-US/Red_Hat_JBoss_Data_Grid/6.2/index.html