Skip to content
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.

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

Integrate pev2 as a component to vizualize execution plan #1431

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MarionGiusti
Copy link
Contributor

No description provided.

@pirlgon
Copy link
Contributor

pirlgon commented Jul 4, 2024

Could we add some text to explain what this does, what tool is used and maybe a link to PEV2? (not sure for the link)
I find the design a bit too minimalist :) .

@pirlgon
Copy link
Contributor

pirlgon commented Jul 4, 2024

I tried a sample plan coming from explain.dalibo.com (Example 5) and the visualisation seems broken. looks like js or css is missing.

@bersace bersace self-requested a review July 22, 2024 12:13
Copy link
Member

@bersace bersace left a comment

Choose a reason for hiding this comment

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

C'est bon pour moi, à terminer avec pierre.

@bersace
Copy link
Member

bersace commented Jul 23, 2024

@marion, pourras-tu ajouter une entrée dans changelog ?

@pgiraud
Copy link
Member

pgiraud commented Jul 23, 2024

@MarionGiusti Can you please rebase your branch? pev2 requires bootstrap to be displayed correctly.

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.

None yet

4 participants