Trivial fix of code comment

Started by Chao Li5 months ago3 messages
#1Chao Li
li.evan.chao@gmail.com
1 attachment(s)

Hi Hackers,

Simple fix "commits" to "commits" in a test spec file. "make check" passed.

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/

Attachments:

v1-0001-Update-comments-in-src-test-isolation-specs-eval-.patchapplication/octet-stream; name=v1-0001-Update-comments-in-src-test-isolation-specs-eval-.patchDownload
From 5624d68d5081076597f159fa8f0cd68c9d2ba03d Mon Sep 17 00:00:00 2001
From: "Chao Li (Evan)" <lic@highgo.com>
Date: Fri, 29 Aug 2025 16:06:09 +0800
Subject: [PATCH v1] Update comments in
 src/test/isolation/specs/eval-plan-qual-trigger.spec

Replaced two instances of "committs" with "commits" to correct a
spelling error in the comments of the isolation test file.

Author: Chao Li <lic@highgo.com>
---
 src/test/isolation/specs/eval-plan-qual-trigger.spec | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/test/isolation/specs/eval-plan-qual-trigger.spec b/src/test/isolation/specs/eval-plan-qual-trigger.spec
index b512edd2879..232b3e27652 100644
--- a/src/test/isolation/specs/eval-plan-qual-trigger.spec
+++ b/src/test/isolation/specs/eval-plan-qual-trigger.spec
@@ -336,7 +336,7 @@ permutation s1_trig_rep_b_u s1_trig_rep_a_u
     s1_ins_a s1_ins_c s1_b_rc s2_b_rc
     s1_upd_a_tob s2_upd_all_data s1_c s2_c
     s0_rep
-# s1 deletes, s2 updates, s1 committs, EPQ failure should lead to no update
+# s1 deletes, s2 updates, s1 commits, EPQ failure should lead to no update
 permutation s1_trig_rep_b_d s1_trig_rep_b_u s1_trig_rep_a_d s1_trig_rep_a_u
     s1_ins_a s1_ins_c s1_b_rc s2_b_rc
     s1_del_a s2_upd_a_data s1_c s2_c
@@ -346,7 +346,7 @@ permutation s1_trig_rep_b_d s1_trig_rep_b_u s1_trig_rep_a_d s1_trig_rep_a_u
     s1_ins_a s1_ins_c s1_b_rc s2_b_rc
     s1_del_a s2_upd_a_data s1_r s2_c
     s0_rep
-# s1 deletes, s2 deletes, s1 committs, EPQ failure should lead to no delete
+# s1 deletes, s2 deletes, s1 commits, EPQ failure should lead to no delete
 permutation s1_trig_rep_b_d s1_trig_rep_a_d
     s1_ins_a s1_ins_c s1_b_rc s2_b_rc
     s1_del_a s2_del_a s1_c s2_c
-- 
2.39.5 (Apple Git-154)

#2Ashutosh Bapat
ashutosh.bapat.oss@gmail.com
In reply to: Chao Li (#1)
Re: Trivial fix of code comment

On Fri, Aug 29, 2025 at 1:45 PM Chao Li <li.evan.chao@gmail.com> wrote:

Hi Hackers,

Simple fix "commits" to "commits" in a test spec file. "make check" passed.

The patch changes "committs" to "commits" which is what it should do.

The change looks good to me.

--
Best Wishes,
Ashutosh Bapat

#3Daniel Gustafsson
daniel@yesql.se
In reply to: Ashutosh Bapat (#2)
Re: Trivial fix of code comment

On 29 Aug 2025, at 13:01, Ashutosh Bapat <ashutosh.bapat.oss@gmail.com> wrote:

On Fri, Aug 29, 2025 at 1:45 PM Chao Li <li.evan.chao@gmail.com> wrote:

Hi Hackers,

Simple fix "commits" to "commits" in a test spec file. "make check" passed.

The patch changes "committs" to "commits" which is what it should do.

Agreed, pushed.

--
Daniel Gustafsson