Skip to content

Don't use select * #70

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

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

Conversation

ndtretyak
Copy link

Name of feature:

Pain or issue this feature alleviates:

All other queries select only required fields, but getAllQry uses SELECT *. I want to add columns to the nosql-created tables and update them via database triggers, but SELECT * prevents this.

Why is this important to the project (if not answered above):

Is there documentation on how to use this feature? If so, where?

In what environments or workflows is this feature supported?

In what environments or workflows is this feature explicitly NOT supported (if any)?

Supporting links/other PRs/issues:

💔Thank you!

@ndtretyak ndtretyak requested a review from a team as a code owner June 17, 2025 15:15
@github-actions github-actions bot added the needs triage Waiting for discussion / prioritization by team label Jun 17, 2025
@hslatman hslatman self-assigned this Jun 17, 2025
@hslatman
Copy link
Member

Hey @ndtretyak, thank you for opening the PR. Our nosql package has been fairly "stable", in that we haven't done a lot of changes lately. We have an outstanding PR to improve the implementation in #53, which would be a better base for your changes. Have you tried running the CA using that branch yet, and does that support your use case?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage Waiting for discussion / prioritization by team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants