Skip to content

[DEVO-1947] upgrade to trino 445#5

Open
JRemitz wants to merge 6 commits into
mainfrom
SRE-5-393e-lts
Open

[DEVO-1947] upgrade to trino 445#5
JRemitz wants to merge 6 commits into
mainfrom
SRE-5-393e-lts

Conversation

@JRemitz

@JRemitz JRemitz commented Jul 26, 2024

Copy link
Copy Markdown
Member
  • Upgrade to Java 21 (LTS)
    • Update sbt version and build script
  • Upgrade to Trino 445
    • Requires modifications to support redesign of SystemAccessControl from upstream

@JRemitz JRemitz requested a review from dmzobel July 26, 2024 22:15
@JRemitz JRemitz self-assigned this Jul 26, 2024
"trino_shared",
"trino_shared_dev"
"trino_shared_dev",
"presto_export",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This appears to be a bug that we've always had? Or something with Scala matching is working different. Not totally sure but queries broke without having this. web reference

@JRemitz JRemitz changed the title [DEVO-adhoc] upgrade to trino 445 [DEVO-1947] upgrade to trino 445 Jul 29, 2024

@dmzobel dmzobel left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approving to unblock you!

@JRemitz

JRemitz commented Aug 13, 2024

Copy link
Copy Markdown
Member Author

Approving to unblock you!

Thanks! I'm actually blocked on the JMX metrics. I removed the canary and am awaiting the public review upstream. They responded and said they'd look at it last week but to no avail yet. 🤞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants