Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Created by
brew bump
Created with
brew bump-formula-pr
.release notes
EXECUTE
doesn't send RowDescriptionThis PR fixes issue connecting to doltgres server using Postgres JDBC driver.
In Postgres docs, it says, "
Execute
doesn't cause ReadyForQuery or RowDescription to be issued."Also, queries that don't return set of rows send
NoData
in response toDescribe
message.Created by the Release workflow to update DoltgreSQL's version
RowDescription
messageRowDescription
message. This includesSELECT
,FETCH
,SHOW
, etc.CommandTag
is now set by theStatementTag
from the parsed query.EmptyQueryResponse
message.Companion PR:
This adds functionality for
CREATE SEQUENCE
andDROP SEQUENCE
. For sequences, this is what is in the PR:This is what is missing:
Of what's missing, merging and
SERIAL
are the main additions, withOWNED BY
required for properSERIAL
support. Table querying is fairly straightforward (thepg_sequences
system table, etc.). The others are, hopefully, unnecessary for now and can be postponed, although perhaps we can save theCACHE
value and ignore it since it's primarily a performance-oriented option.With that,
SERIAL
and merging will come in the next PR, since this is already very large. Also of note, theWriter
andReader
utilities were implemented for my Convert-to-Cast PR but they were useful here, so I pulled them from that in-flight PR and added them here.Closed Issues