Skip to content

[sqlite] Please add possibility to get prepared statement columns #57457

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.

Sign up for GitHub

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

Jump to bottom
Closed
zdm opened this issue Mar 14, 2025 · 1 comment · Fixed by #57490
Closed

[sqlite] Please add possibility to get prepared statement columns #57457

zdm opened this issue Mar 14, 2025 · 1 comment · Fixed by #57490
Labels
feature request Issues that request new features to be added to Node.js. sqlite Issues and PRs related to the SQLite subsystem.

Comments

@zdm
Copy link

zdm commented Mar 14, 2025

What is the problem this feature will solve?

Our sqlite driver uses columns into to automatically decode content.
In any case this is useful functionality that can be used in many other cases (debugging, logging, etc.).

What is the feature you are proposing to solve the problem?

I am not familiar with the sqlite internal API but I think you need to expose sth.columns() method or something similar.

What alternatives have you considered?

No alternatives.
@zdm zdm added the feature request Issues that request new features to be added to Node.js. label Mar 14, 2025
@github-project-automation github-project-automation bot moved this to Awaiting Triage in Node.js feature requests Mar 14, 2025
@cjihrig cjihrig added the sqlite Issues and PRs related to the SQLite subsystem. label Mar 14, 2025
cjihrig added a commit to cjihrig/node that referenced this issue Mar 16, 2025
@cjihrig
sqlite: add StatementSync.prototype.columns()
39ce88d 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: nodejs#57457
@cjihrig cjihrig mentioned this issue Mar 16, 2025
Merged
@cjihrig
Copy link
Contributor

cjihrig commented Mar 16, 2025

Addressed in #57490.

cjihrig added a commit to cjihrig/node that referenced this issue Mar 17, 2025
@cjihrig
sqlite: add StatementSync.prototype.columns()
0875ac7 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: nodejs#57457
cjihrig added a commit to cjihrig/node that referenced this issue Mar 17, 2025
@cjihrig
sqlite: add StatementSync.prototype.columns()
7928ac7 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: nodejs#57457
nodejs-github-bot pushed a commit that referenced this issue Mar 18, 2025
@cjihrig
sqlite: add StatementSync.prototype.columns()
1de917b 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
aduh95 pushed a commit that referenced this issue Mar 23, 2025
@cjihrig @aduh95
sqlite: add StatementSync.prototype.columns()
4b04c92 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
RafaelGSS pushed a commit that referenced this issue Apr 1, 2025
@cjihrig @RafaelGSS
sqlite: add StatementSync.prototype.columns()
be8bcac 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
RafaelGSS pushed a commit that referenced this issue Apr 1, 2025
@cjihrig @RafaelGSS
sqlite: add StatementSync.prototype.columns()
d4efb85 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
RafaelGSS pushed a commit to RafaelGSS/node that referenced this issue Apr 8, 2025
@cjihrig @RafaelGSS
sqlite: add StatementSync.prototype.columns()
0c5ec17 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: nodejs#57457
PR-URL: nodejs#57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
RafaelGSS pushed a commit that referenced this issue Apr 14, 2025
@cjihrig @RafaelGSS
sqlite: add StatementSync.prototype.columns()
ba5e7fa 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
RafaelGSS pushed a commit that referenced this issue Apr 14, 2025
@cjihrig @RafaelGSS
sqlite: add StatementSync.prototype.columns()
f33f040 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
RafaelGSS pushed a commit that referenced this issue May 1, 2025
@cjihrig @RafaelGSS
sqlite: add StatementSync.prototype.columns()
ff6cc92 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
RafaelGSS pushed a commit that referenced this issue May 2, 2025
@cjihrig @RafaelGSS
sqlite: add StatementSync.prototype.columns()
dd38a47 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
aduh95 pushed a commit that referenced this issue May 6, 2025
@cjihrig @aduh95
sqlite: add StatementSync.prototype.columns()
88cb6fd 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
aduh95 pushed a commit that referenced this issue May 6, 2025
@cjihrig @aduh95
sqlite: add StatementSync.prototype.columns()
e0c8c16 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
RafaelGSS pushed a commit that referenced this issue May 14, 2025
@cjihrig @RafaelGSS
sqlite: add StatementSync.prototype.columns()
acb52da 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
aduh95 pushed a commit that referenced this issue May 16, 2025
@cjihrig @aduh95
sqlite: add StatementSync.prototype.columns()
fa8fcba 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
aduh95 pushed a commit that referenced this issue May 17, 2025
@cjihrig @aduh95
sqlite: add StatementSync.prototype.columns()
813cc34 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
aduh95 pushed a commit that referenced this issue May 17, 2025
@cjihrig @aduh95
sqlite: add StatementSync.prototype.columns()
7ec6d5f 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
aduh95 pushed a commit that referenced this issue May 17, 2025
@cjihrig @aduh95
sqlite: add StatementSync.prototype.columns()
2dbadb8 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
aduh95 pushed a commit that referenced this issue May 18, 2025
@cjihrig @aduh95
sqlite: add StatementSync.prototype.columns()
2a2fc76 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
aduh95 pushed a commit that referenced this issue May 19, 2025
@cjihrig @aduh95
sqlite: add StatementSync.prototype.columns()
c510391 
This commit adds a method for retrieving column metadata from
a prepared statement.

Fixes: #57457
PR-URL: #57490
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Edy Silva <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request Issues that request new features to be added to Node.js. sqlite Issues and PRs related to the SQLite subsystem.
Projects
Node.js feature requests
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

sqlite: add StatementSync.prototype.columns() cjihrig/node
2 participants
@zdm @cjihrig