Skip to content

Release 6.19.0 arenadata31#296

Merged
Stolb27 merged 2 commits intoadb-6.xfrom
6.19.0_arenadata31
Jan 20, 2022
Merged

Release 6.19.0 arenadata31#296
Stolb27 merged 2 commits intoadb-6.xfrom
6.19.0_arenadata31

Conversation

@Stolb27
Copy link
Collaborator

@Stolb27 Stolb27 commented Jan 20, 2022

InnerLife0 and others added 2 commits January 20, 2022 20:40
… output list when using ORCA. (#284)

The changes introduced in PR #12176 revealed a problem with excessive Bitmap Heap Scan's targetlist which includes Recheck and Filter Condition's columns (introduced in old 97edf9b).
Memory tuple binding generated based on targetlist with different count of attrs may point to different offsets and so, to some garbage or out of range memory area. See issue #12796 for more.
Additional regression test shows how to reproduce a bug before fix. A couple of tests was modified, because there is no need of projection in them started from this patch.
Natively raise exception in case of targetlist mismatch.
GUC value rollback was added by commit 09e4d0a

But it was found that GUC values of PGC_STRING type should be
quoted or parsing will fail otherwise as, for example, for
gp_default_storage_options GUC:

"ERROR","42601","syntax error at or near ""=""",,,,,,"SET gp_default_storage_options TO appendonly=false,blocksize=32768,compresstype=none,checksum=true,orientation=row",45,,"scan.l",1062,

As failed SET command leads to transaction abort that leads to
backend exit that triggers RemoveTempRelationsCallback to remove
temp relations, the new tests verify that after GUC PGC_STRING value
rollback no temp relation is removed.

The values with GUC_LIST_INPUT set in flags should not be quoted as
it will destroy list otherwise.
For example as for search_path:
SET search_path TO '"$user",public'
should remain
SET search_path TO "$user",public
@Stolb27 Stolb27 enabled auto-merge January 20, 2022 10:44
@Stolb27 Stolb27 requested a review from a team January 20, 2022 10:45
Copy link

@InnerLife0 InnerLife0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mine is OK.

@Stolb27 Stolb27 merged commit 587a3f7 into adb-6.x Jan 20, 2022
@Stolb27 Stolb27 deleted the 6.19.0_arenadata31 branch January 20, 2022 13:58
RekGRpth pushed a commit that referenced this pull request Mar 4, 2026
* Retarget Behave tests CI to v24 tag
* Mount sqldump/dump.sql to cdw service in docker compose
* Enable gpexpand behave test

Task: ADBDEV-9130
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants