This is an automated email from the ASF dual-hosted git repository.

reshke pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/cloudberry.git


The following commit(s) were added to refs/heads/main by this push:
     new cfba97aa735 Remove bogus loop in single-iteration code (#1580)
cfba97aa735 is described below

commit cfba97aa7354892f1d9780f0a6983a58c1619830
Author: reshke <[email protected]>
AuthorDate: Wed Feb 25 10:42:43 2026 +0500

    Remove bogus loop in single-iteration code (#1580)
    
    Spotted while casually reading SonarQube output. Commit simply removes loop 
in oddly-fashioned code.
    
    Reviewed-by: Andrey Borodin <[email protected]>
    Reviewed-by: Jianghua Yang <[email protected]>
---
 src/backend/access/appendonly/appendonlyam.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/backend/access/appendonly/appendonlyam.c 
b/src/backend/access/appendonly/appendonlyam.c
index b5f6a17a597..defa5ccc80e 100755
--- a/src/backend/access/appendonly/appendonlyam.c
+++ b/src/backend/access/appendonly/appendonlyam.c
@@ -1983,7 +1983,7 @@ appendonly_endscan(TableScanDesc scan)
 static pg_attribute_hot_inline bool
 appendonly_getnextslot_noqual(AppendOnlyScanDesc aoscan, ScanDirection 
direction, TupleTableSlot *slot)
 {
-       while (appendonlygettup(aoscan, direction, aoscan->rs_base.rs_nkeys, 
aoscan->aos_key, slot))
+       if (appendonlygettup(aoscan, direction, aoscan->rs_base.rs_nkeys, 
aoscan->aos_key, slot))
        {
                pgstat_count_heap_getnext(aoscan->aos_rd);
                return true;


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to