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

Custom binary platforms for query engine #343

Closed
divyenduz opened this issue Aug 9, 2019 · 12 comments
Closed

Custom binary platforms for query engine #343

divyenduz opened this issue Aug 9, 2019 · 12 comments
Assignees
Labels
kind/feature A request for a new feature.
Milestone

Comments

@divyenduz
Copy link
Contributor

Recently, we implemented most parts of this spec.

Custom binaries (manually compiled) section is still to be implemented though.

@divyenduz divyenduz added kind/feature A request for a new feature. area/cli labels Aug 9, 2019
@janpio janpio removed the area/cli label Aug 9, 2019
@janpio janpio added this to the Preview 7 milestone Aug 9, 2019
@janpio janpio assigned janpio and timsuchanek and unassigned janpio Aug 9, 2019
@janpio janpio removed the candidate label Aug 9, 2019
@sorenbs sorenbs mentioned this issue Aug 12, 2019
@janpio janpio added backlog and removed backlog labels Aug 12, 2019
@janpio janpio removed this from the Preview 7 milestone Aug 15, 2019
@janpio janpio added this to the Preview 8 milestone Aug 15, 2019
@janpio janpio removed the candidate label Aug 15, 2019
@mavilein
Copy link
Member

The required part in the Prisma schema parser is done.

@janpio
Copy link
Member

janpio commented Aug 22, 2019

Sync between @mavilein and @timsuchanek necessary.
Spec is not "finished enough" to start implementation, ping @divyenduz

@janpio janpio removed this from the Preview 8 milestone Aug 22, 2019
@janpio janpio removed the candidate label Aug 23, 2019
@divyenduz
Copy link
Contributor Author

The spec is ready now to be implemented: https://github.com/prisma/specs/tree/master/binaries#how-to-fetch-binaries-1

@divyenduz divyenduz removed their assignment Aug 28, 2019
@janpio janpio added this to the Preview 10 milestone Aug 30, 2019
@divyenduz
Copy link
Contributor Author

This majorly impacted the following:

  1. Windows users (other users which we did not support yet) -> Windows the major use case is supported now
  2. platforms -> binaryTargets rename -> This was suggested in the spec to avoid SEO wars with a future product that might include platforms. That is tracked here: Rename platforms to binaryTargets #553

This reduces the priority of this issue for now.

@janpio
Copy link
Member

janpio commented Sep 19, 2019

Internal note: Dropped as re-prioritized.

@janpio janpio removed this from the Preview 12 milestone Sep 20, 2019
@janpio
Copy link
Member

janpio commented Sep 27, 2019

What exactly is not implemented here @divyenduz? ENV vars like PRISMA_QUERY_ENGINE_BINARY or the compiling part?

@divyenduz
Copy link
Contributor Author

Yes, env vars like PRISMA_QUERY_ENGINE_BINARY are tracked by this issue, how to compile a custom binary is already documented (maybe?) I remember doing it but can't find it anymore, so maybe a separate issue for that.

Renaming (a subset of this issue), is tracked here: #553

@janpio
Copy link
Member

janpio commented Sep 27, 2019

Can you try to apply https://github.com/prisma/specs/blob/master/meta/specs.md#implemented-vs-not-implemented to the spec then for that? Thanks.

@timsuchanek
Copy link
Contributor

Released in the latest preview.

@divyenduz
Copy link
Contributor Author

The variables PRISMA_QUERY_ENGINE_BINARY and PRISMA_MIGRATION_ENGINE_BINARY are ignored by the CLI.

I validated this by providing an incorrect path, it should have thrown and error but it worked instead.

@janpio
Copy link
Member

janpio commented Nov 29, 2019

Please open a new bug report for this @divyenduz instead of reopening the feature issue.

@pantharshit00
Copy link
Contributor

There is already an issue about this: #1034

I am closing this one

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

5 participants