Conversation
|
Huge thanks to @grooverdan ! It provides a great reference. Haibo and I will review it ASAP. |
|
Great work! Thanks @grooverdan . The one-command setup makes it perfect for quick testing and demos. |
|
@grooverdan Hi Daniel, thanks for your great PR. Your docker-compose stack is a really nice end-to-end example (bringing up all components and validating Sorry for the delayed reply — while reproducing this PR on Debian + Docker, we ran into (and spent some time chasing down) a mismatch between The root cause is a stats-refresh issue during metadata collection: if we run We’ve opened PR #81 to address this by running |
Agree — I also think In the spirit of https://jira.mariadb.org/browse/MDEV-38409 (make VIDEX easy to install/start), as the next step, we plan to provide a dedicated
|
|
Great. I'll leave this on hold until there's a published container image and then rework this as an example to use the published container image. |
This removes the tar layer for a single file.
This shows how a MariaDB instance is created, and videx uses the server implementation whcih is populated from the instance data. A videx plugin is used in a second instance that forms the virtual plans.
428d3ca to
5650b81
Compare
|
updated with env file and test container |
Pull Request Summary
Add a MariaDB compose file to show how VIDEX and MariaDB components are related and the steps required.
Detailed Description
With videx as a plugin as a MariaDB now, I was looking at how to make use of it. I thought a compose example that included a container of just the VIDEX server would show the steps that need to occur.
With single service per function, a user can run the explain in the mariadb container and the videx plugin enabled mariadb container for comparison.
I attempted to use
tpch_tiny.sql.tar.gzdirectly however MariaDB container uses atar.gzfile as an indicator that its a MariaDB backup that is attempted to be restored.I think that
build/Dockerfile.videxservershould be a separately published container rather than a build in compose, but feedback welcome on this.I probably could factor the compose file env variables rather than having multiple copies of the same thing (passwords, database names..) in multiple services.
Important: Before submitting, please complete the description above and review the checklist below.
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
By submitting this PR, you confirm that you've read these guidelines and your changes align with the project's contribution standards.