Move formatting out from ParquetCorruptionException#19907
Move formatting out from ParquetCorruptionException#19907ksobolew wants to merge 1 commit intotrinodb:masterfrom
Conversation
core/trino-main/src/main/java/io/trino/sql/analyzer/ExpressionAnalyzer.java
Outdated
Show resolved
Hide resolved
ksobolew
force-pushed
the
kudi/error-prone-annotate-format-method-remnants
branch
from
3089cdd to
27b5387
Compare
core/trino-main/src/main/java/io/trino/execution/SetSessionTask.java
Outdated
Show resolved
Hide resolved
lib/trino-parquet/src/main/java/io/trino/parquet/ParquetCorruptionException.java
Outdated
Show resolved
Hide resolved
|
CI: #16315 |
ksobolew
force-pushed
the
kudi/error-prone-annotate-format-method-remnants
branch
from
27b5387 to
edd6722
Compare
ksobolew
changed the title
|
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua |
|
👋 @ksobolew I assume you will continue work on this PR and will leave it open. |
|
@mosabua I don't know, from my point of view it's "done", but perhaps it doesn't bring enough value. |
|
@findepi you already reviewed a bit. Whats your opinion .. complete to merge or close? |
...to its callers, to make the exception class more focused.
ksobolew
force-pushed
the
kudi/error-prone-annotate-format-method-remnants
branch
from
edd6722 to
f45f211
Compare
ksobolew
changed the title
|
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua |
|
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua |
|
Closing this pull request, as it has been stale for six weeks. Feel free to re-open at any time. |
|
@findepi I am leaving this closed .. feel free to reopen and merge if its good to go |
3 participants
Description
...to its callers, to make the exception class more focused. This in fact makes it consistent with how other exception classes were transformed, though I don't remember the exact commits right now.
Additional context and related issues
These are the remnants of #14933 and #15963 (and kind of #15052 too) after #19899 (specifically f3c803e) made them mostly obsolete.
Release notes
(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text: