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

All Engines should print commit hash on startup #35

Closed
mavilein opened this issue Sep 6, 2019 · 7 comments
Closed

All Engines should print commit hash on startup #35

mavilein opened this issue Sep 6, 2019 · 7 comments
Assignees
Labels
kind/feature A request for a new feature.
Milestone

Comments

@mavilein
Copy link
Member

mavilein commented Sep 6, 2019

@timsuchanek wants to use this to validate that the javascript code and downloaded engines are in sync. We often had the case that this did not work properly and debugging is unecessarily hard.

@mavilein mavilein added this to the Preview 11 milestone Sep 6, 2019
@janpio janpio added kind/feature A request for a new feature. process/next-milestone labels Sep 12, 2019
@janpio janpio modified the milestones: Preview 11, Preview 12 Sep 12, 2019
@janpio janpio removed this from the Preview 12 milestone Sep 20, 2019
@pimeys
Copy link
Contributor

pimeys commented Oct 7, 2019

We have this now when querying /server_info from the query-engine runtime.

@janpio janpio added this to the Preview 14 milestone Oct 7, 2019
@pimeys
Copy link
Contributor

pimeys commented Oct 7, 2019

We have this also in the command line parameters for query and migration engines:

cargo run --bin prisma -- --version
2b78c35cf302bf80ae5aaa863c5eee0d09541f83
cargo run --bin migration-engine -- --version
2b78c35cf302bf80ae5aaa863c5eee0d09541f83

@pimeys pimeys closed this as completed Oct 7, 2019
@pimeys
Copy link
Contributor

pimeys commented Oct 7, 2019

fe8f5f7

@janpio
Copy link
Member

janpio commented Oct 8, 2019

How will this be implemented/used?

@pimeys
Copy link
Contributor

pimeys commented Oct 8, 2019

You just run the binary with --version parameter.

@janpio
Copy link
Member

janpio commented Oct 8, 2019

Ha, yes on the engine side - I was wondering where and how this would be used by @timsuchanek to do what exactly.

@pimeys
Copy link
Contributor

pimeys commented Oct 9, 2019 via email

miguelff pushed a commit that referenced this issue Apr 20, 2023
Clarify the meaning of schema_name on ConnectionInfo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature A request for a new feature.
Projects
None yet
Development

No branches or pull requests

3 participants