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
The parser needs to be built before
go run -mod=readonly ./utils/genminver_validation/ $FILE
Fixes The LIMIT expression is not working dolthub/doltgresql#172
Companion:
This is Doltgres' implementation of
RootValue
. For now, it's essentially taken wholesale from Dolt as a starting point. Of note, we probably want to at least get something different for our collation values sometime soon. Also, we're using the same serial identifier for root values between Doltgres and Dolt. This simplifies a good bit of code in Dolt, so taking on the naming requirement seems well worth it.name
, fix bool -> name castMove the main package to cmd/doltgres so that the source built binary matches the released binary.
xid
typename
typeThis replaces all of the
init()
functions throughout the project with a package-levelInit()
function that is called from a newinitialization
package. This now gives an explicit ordering to the calls. The biggest disadvantage of this approach is that some packages cannot have package-level tests that can access package-private variables without potentially causing import cycles if they need to initialize another package. This was also a limitation with the previousinit()
approach as well, but the vast majority of our tests are done from thetesting
subdirectories (in part due to this limitation), so we've not lost anything by changing to this approach.Things that work:
Create schema
Things that don't work:
public
schema for new databasesReverts dolthub/doltgresql#176 as these actions have been updated and work now
Created by the Release workflow to update DoltgreSQL's version
CREATE VIEW
Depends on:
Closed Issues