Skip to content

Conversation

@samwillis
Copy link
Collaborator

No description provided.

@samwillis samwillis force-pushed the samwillis/pglite-socket branch from a9d02f4 to 7f64066 Compare April 12, 2025 14:19
@samwillis samwillis force-pushed the samwillis/pglite-socket branch from 88fa08d to 38c7f45 Compare April 13, 2025 16:07
@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2025

@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2025

@samwillis samwillis changed the title WIP feat(pglite-socket): socket server for PGlite feat(pglite-socket): socket server for PGlite May 8, 2025
@pmp-p
Copy link
Contributor

pmp-p commented May 9, 2025

to please some wire protocol decoders, we'll skip extra ready for query in pipeline mode for:
pg_proto.c: case 'P': /* PqMsg_Parse : exec_parse_message /
pg_proto.c: case 'B': /
PqMsg_Bind : exec_bind_message /
pg_proto.c: case 'E': /
PqMsg_Execute : execute /
pg_proto.c: case 'C': /
PqMsg_Close / PqMsg_CommandComplete : close pg_proto.c: case 'D': /* PqMsg_Describe : describe */

We may skip in other mode as we discover other needs from established wire libraries
If a keep alive is needed then an empty (3) may possibly be used.

@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2025

@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2025

@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2025

@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2025

@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2025

@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2025

@github-actions
Copy link
Contributor

@github-actions
Copy link
Contributor

@pmp-p pmp-p linked an issue May 12, 2025 that may be closed by this pull request
@pmp-p pmp-p self-requested a review May 12, 2025 12:01
Copy link
Contributor

@pmp-p pmp-p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm !

Copy link
Collaborator

@tdrz tdrz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@tdrz tdrz merged commit 839bb5a into main May 12, 2025
1 of 2 checks passed
@tdrz tdrz deleted the samwillis/pglite-socket branch May 12, 2025 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

epic Include in the roadmap.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't seem to use prepared statements using libpq

4 participants