Skip to content

Remove generated test file misc.sql#1600

Merged
leborchuk merged 1 commit intoapache:mainfrom
gfphoenix78:fix-test-path
Mar 3, 2026
Merged

Remove generated test file misc.sql#1600
leborchuk merged 1 commit intoapache:mainfrom
gfphoenix78:fix-test-path

Conversation

@gfphoenix78
Copy link
Contributor

src/test/regress/sql/misc.sql is generated by
src/test/regress/input/misc.souce, it should not be add in sql directory.

Fixes #ISSUE_Number

What does this PR do?

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


@avamingli
Copy link
Contributor

Thanks for this fix! #1592 is the reason.

Copy link
Contributor

@avamingli avamingli left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Contributor

@leborchuk leborchuk left a comment

Choose a reason for hiding this comment

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

Yep, it should be deleted, sorry I am not fully online to fix it fast enough

src/test/regress/sql/misc.sql is generated by
src/test/regress/input/misc.souce, it should not be add in sql directory.
@leborchuk leborchuk merged commit 64c879a into apache:main Mar 3, 2026
41 checks passed
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.

4 participants