Thanks for the review.
I agree with you.
I will wait for Daogang's feedback and right after change everything you requested and
On 09/09/2010 11:35 AM, Libing Song wrote:
> Hi Alfranio,
> Nice work!
> Please find my review comments below.
> Not Approved.
> REQUIRED CHANGES
> +# B Sn N Cn/Rn C/R
> It should be
> +# B T Sn N Cn/Rn C/R
> Please check all other comments.
> RE2. Could you please comment(In both WL and patch) in which sequence
> the savepoints is rolled back.
> B T Sn N Sn T N Rn Rn C
> means: B T S1 N S2 T N R2 R1 C.
> It rolls back S2 first and then rolls back S1.
> RE3. You forgot the test for 'B T Sn TEMP Cn/Rn C/R'
> RE4. Could you please comment(In both WL and patch) the expect behavior
> for each test case?
> I think 'B T S1 N S2 T N R2 R1 C' is more clear.
> So I suggest to use it.