Skip to content

Conversation

@YoungHypo
Copy link
Contributor

Pull Request Summary

This ci pipeline was accidentally deleted. The ci result of current main brach is https://github.com/YoungHypo/videx/actions/runs/19622613467/job/56185678200

Related PR

#40


Contribution Guidelines (Expand for Details)

We appreciate your contribution to VIDEX! To ensure a smooth review process and maintain high code quality, please adhere to the following guidelines:

Pull Request Title Format

Your PR title should start with one of these prefixes to indicate the nature of the change:

  • [Core]: Changes to core engine functionality
  • [Opt]: Changes to VIDEX-Optimizer-Plugin
  • [Stats]: Changes to VIDEX-Statistic-Server
  • [Algo]: Implementation of new algorithms for NDV, cardinality estimation, etc.
  • [Pipe]: Enhancements to the pipeline (e.g., data collection, environment setup)
  • [Bug]: Corrections to existing functionality
  • [CI]: Changes to build process or CI pipeline
  • [Docs]: Updates or additions to documentation
  • [Test]: Adding or updating tests
  • [Perf]: Performance improvements
  • [Misc]: For changes not covered above (use sparingly)

Note: For changes spanning multiple categories, use the most specific prefix or multiple prefixes in order of importance (e.g., [Algorithm][Stats]).

Submission Checklist

  • PR title includes appropriate prefix(es)
  • Changes are clearly explained in the PR description
  • New and existing tests pass successfully
  • Code adheres to project style and best practices
  • Documentation updated to reflect changes (if applicable)
  • Changes have been tested on both Plugin-Mode and Standalone-Mode (if applicable)
  • Statistical accuracy has been verified (for algorithm or optimizer changes)
  • No regression in query plan accuracy compared to InnoDB (if applicable)
  • Performance benchmarks conducted (for performance-sensitive changes)

By submitting this PR, you confirm that you've read these guidelines and your changes align with the project's contribution standards.

@kr11
Copy link
Collaborator

kr11 commented Nov 24, 2025

Great PR, haibo! I will review this PR in this week.

@kr11 kr11 self-requested a review November 24, 2025 06:29
@kr11 kr11 mentioned this pull request Nov 24, 2025
@kr11 kr11 added the test/ci/cd about test, ci, cd label Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test/ci/cd about test, ci, cd

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants