-
Notifications
You must be signed in to change notification settings - Fork 16
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
Bump pg_lakehouse to the latest release #741
Conversation
This pull request is automatically being deployed by Amplify Hosting (learn more). |
0c3d9f7
to
5757af4
Compare
Learning not to "tidy up" older version numbers in my pushes here. |
5757af4
to
45bb409
Compare
@@ -15,17 +15,17 @@ RUN apt-get update && apt-get install -y \ | |||
RUN curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y | |||
|
|||
# Set default Rust version | |||
RUN /root/.cargo/bin/rustup default stable | |||
RUN /root/.cargo/bin/rustup default 1.78.0-x86_64-unknown-linux-gnu |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At least one other extension does this already, so doing to as a quick workaround seemed fine rather than figuring out how to update the pgrx base image.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should also update the base pgrx image tho, yes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but I touch this repo infrequently and as 1.8.0 only came out two weeks ago I wasn't in a rush to figure it out.
But yeah, it might be nice if these builders didn't break when new rust versions come out.
@@ -15,17 +15,17 @@ RUN apt-get update && apt-get install -y \ | |||
RUN curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y | |||
|
|||
# Set default Rust version | |||
RUN /root/.cargo/bin/rustup default stable | |||
RUN /root/.cargo/bin/rustup default 1.78.0-x86_64-unknown-linux-gnu |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should also update the base pgrx image tho, yes?
v0.9.0 was released yesterday and some new functions are used in the documentation, etc.
45bb409
to
0dfe213
Compare
v0.9.0 was released yesterday and some new functions are used in the documentation, etc.
Getting the latest version in trunk for our use.