Skip to content

test: add ST_ISVALID geometry function test cases #46234

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
zhuwenxing wants to merge 1 commit into
base: 2.6
Choose a base branch
from
Open

test: add ST_ISVALID geometry function test cases #46234

zhuwenxing wants to merge 1 commit into from

Conversation

@zhuwenxing
Copy link
Contributor

@zhuwenxing zhuwenxing commented Dec 9, 2025

master pr: #46232

@zhuwenxing zhuwenxing added this to the 2.6.8 milestone Dec 9, 2025
@sre-ci-robot
Copy link
Contributor

sre-ci-robot commented Dec 9, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: zhuwenxing
To complete the pull request process, please assign xiaofan-luan after the PR has been reviewed.
You can assign the PR to them by writing /assign @xiaofan-luan in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sre-ci-robot sre-ci-robot added the size/L Denotes a PR that changes 100-499 lines. label Dec 9, 2025
@mergify mergify bot added dco-passed DCO check passed. kind/test labels Dec 9, 2025
@sre-ci-robot
Copy link
Contributor

sre-ci-robot commented Dec 9, 2025

[ci-v2-notice]
Notice: We are gradually rolling out the new ci-v2 system.

  • Legacy CI jobs remain unaffected, you can just ignore ci-v2 if you don't want to run it.
  • Additional "ci-v2/*" checkers will run for this PR to ensure the new ci-v2 system is working as expected.
  • For tests that exist in both v1 and v2, passing in either system is considered PASS.

To rerun ci-v2 checks, comment with:

  • /ci-rerun-code-check // for ci-v2/code-check
  • /ci-rerun-build // for ci-v2/build
  • /ci-rerun-ut-integration // for ci-v2/ut-integration
  • /ci-rerun-ut-go // for ci-v2/ut-go
  • /ci-rerun-ut-cpp // for ci-v2/ut-cpp
  • /ci-rerun-ut // for all ci-v2/ut-integration, ci-v2/ut-go, ci-v2/ut-cpp
  • /ci-rerun-e2e-arm // for ci-v2/e2e-arm [master branch only]
  • /ci-rerun-e2e-default // for ci-v2/e2e-default [master branch only]

If you have any questions or requests, please contact @zhikunyao.

@sre-ci-robot sre-ci-robot added the do-not-merge/need-merge-master-first any pr merge to release branch need to merge master first label Dec 9, 2025
@sre-ci-robot
Copy link
Contributor

sre-ci-robot commented Dec 9, 2025

[INFO] PR Label Summary by Default
[FAILED] PR #46232 not merged

Use /refresh-label to update related check and label manually

@mergify
Copy link
Contributor

mergify bot commented Dec 9, 2025

@zhuwenxing cpu-e2e job failed, comment /run-cpu-e2e can trigger the job again.

@zhuwenxing
Copy link
Contributor Author

zhuwenxing commented Dec 10, 2025

/hold

feature pr not cp into 2.6

@zhuwenxing
Copy link
Contributor Author

zhuwenxing commented Dec 30, 2025

/unhold

@sre-ci-robot sre-ci-robot removed do-not-merge/hold do-not-merge/need-merge-master-first any pr merge to release branch need to merge master first labels Dec 30, 2025
@sre-ci-robot
Copy link
Contributor

sre-ci-robot commented Dec 30, 2025

[INFO] PR Label Summary by Default
[SUCCESS] PR #46232 merged to master

Use /refresh-label to update related check and label manually

@yanliang567 yanliang567 modified the milestones: 2.6.8, 2.6.9 Jan 5, 2026
@yanliang567 yanliang567 modified the milestones: 2.6.9, 2.6.10 Jan 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@binbinlv binbinlv Awaiting requested review from binbinlv

@yanliang567 yanliang567 Awaiting requested review from yanliang567

Assignees

No one assigned

Labels

area/test dco-passed DCO check passed. kind/test sig/testing size/L Denotes a PR that changes 100-499 lines.

Projects

None yet

Milestone

2.6.10

Development

Successfully merging this pull request may close these issues.

3 participants

@zhuwenxing @sre-ci-robot @yanliang567