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

Get rid of SELECT * FROM * #1220

Open
Centril opened this issue May 13, 2024 · 1 comment
Open

Get rid of SELECT * FROM * #1220

Centril opened this issue May 13, 2024 · 1 comment
Labels
api-break A PR that breaks some user-visible API

Comments

@Centril
Copy link
Contributor

Centril commented May 13, 2024

This issue is meant to track the eventual removal of SELECT * FROM * because as @gefjon says, it is deeply cursed and evil.

@Centril Centril added the api-break A PR that breaks some user-visible API label May 13, 2024
@gefjon
Copy link
Contributor

gefjon commented May 13, 2024

More specifically:

  • It is a weird non-SQL fragment.
  • We have to decide where it is and isn't valid: can I spacetime sql my-module 'SELECT * FROM *' from the command-line? Can I issue it as a one-off query? Can it coexist with other queries in a subscription?
  • We want to encourage client authors to subscribe only to the data they need, but SELECT * FROM * encourages the opposite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-break A PR that breaks some user-visible API
Projects
None yet
Development

No branches or pull requests

2 participants