-
Notifications
You must be signed in to change notification settings - Fork 198
CpsFlowExecution: parseScript(): log "Method Too Large" situations more readably #817
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
Open
jimklimov
wants to merge
53
commits into
jenkinsci:master
Choose a base branch
from
jimklimov:logging-exMTL
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
390427a
CpsFlowExecution: parseScript(): log "Method Too Large" situations mo…
jimklimov c438ebd
Merge branch 'master' into logging-exMTL
jimklimov 93d4621
Merge remote-tracking branch 'upstream/master' into logging-exMTL
jimklimov b91c7f6
CpsFlowExecution: import groovyjarjarasm.asm.MethodTooLargeException …
jimklimov 28cb0c3
CpsScriptTest: add methodTooLargeExceptionFabricated() and methodTooL…
jimklimov df2b78f
CpsFlowExecution, CpsScriptTest: handle also MultipleCompilationError…
jimklimov ffe1f1a
CpsScriptTest.methodTooLargeExceptionRealistic(): reduce iteration co…
jimklimov fa95de7
CpsFlowExecution.parseScript(): handle MethodTooLargeException withou…
jimklimov 66afa75
CpsFlowExecution.parseScript(): MethodTooLargeException: actionable s…
jimklimov 74a125c
CpsScriptTest.methodTooLargeExceptionRealistic(): update comments and…
jimklimov 5a50a57
CpsScriptTest.methodTooLargeExceptionRealistic(): decouple maxStagesM…
jimklimov d781a3b
Merge branch 'master' into logging-exMTL
jimklimov 89bbd40
Update plugin/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsFlo…
jimklimov 38cc476
Merge branch 'master' into logging-exMTL
jimklimov 0fc23a5
Merge branch 'master' into logging-exMTL
jimklimov 92aadc9
CpsFlowExecution: restore the multi-catch for "RuntimeException | Error"
jimklimov 8c36e27
Revert "Update plugin/src/main/java/org/jenkinsci/plugins/workflow/cp…
jimklimov 82470d2
CpsFlowExecution: restore the write into server log, but only at FINE…
jimklimov db51a8a
CpsFlowExecution: comment that the remaining case is "ecCount == 1 ex…
jimklimov 20213ea
CpsFlowExecution: recognize also CpsCompilationErrorsException for Me…
jimklimov f23b610
CpsFlowExecution, CpsScriptTest: recognize also Groovy (JSL step or c…
jimklimov 21a3f8d
CpsFlowExecution: introduce METHOD_TOO_LARGE_LOGGER for separated log…
jimklimov 47a5e84
CpsFlowExecution: annotate what we post into METHOD_TOO_LARGE_LOGGER
jimklimov 2adbecf
CpsFlowExecution: refactor whole reportSuspectedMethodTooLarge() magi…
jimklimov 5954624
CpsFlowExecution: parseScript(): IDEA complains for "throw reportSusp…
jimklimov dac4ca6
CpsScript: getProperty(): involve CpsFlowExecution.reportSuspectedMet…
jimklimov 4126b11
CpsFlowExecution: reportSuspectedMethodTooLarge(): fix indentations a…
jimklimov b078e7a
CpsFlowExecution: reportSuspectedMethodTooLarge(): when returning sho…
jimklimov b02afc3
CpsFlowExecution: reportSuspectedMethodTooLarge(): xStr should not in…
jimklimov 5336cbe
CpsFlowExecution: reportSuspectedMethodTooLarge(): at least CpsCompil…
jimklimov 4e19e1f
CpsFlowExecution: reportSuspectedMethodTooLarge(): describe what we s…
jimklimov 93adf33
CpsFlowExecution: reportSuspectedMethodTooLarge(): document MTLE_CLAS…
jimklimov a512368
CpsFlowExecution: reportSuspectedMethodTooLarge(): overflowedClassNam…
jimklimov bfa34cb
CpsFlowExecution: reportSuspectedMethodTooLarge(): support also pipel…
jimklimov 2fd7299
CpsFlowExecution: reportSuspectedMethodTooLarge(): wrap original log …
jimklimov c5e79da
CpsFlowExecution: reportSuspectedMethodTooLarge(): refactor actionabl…
jimklimov 4d39666
CpsFlowExecution: reportSuspectedMethodTooLarge(): refactor overflowe…
jimklimov 4f5ab70
CpsFlowExecution: reportSuspectedMethodTooLarge(): match by *short* o…
jimklimov 9491ce1
CpsFlowExecution: reportSuspectedMethodTooLarge(): wrap original log …
jimklimov 99362bc
CpsFlowExecution: reportSuspectedMethodTooLarge(): comment about "int…
jimklimov 343c5da
CpsFlowExecution: reportSuspectedMethodTooLarge(): do not hijack mtlE…
jimklimov 82edfc1
CpsFlowExecution: reportSuspectedMethodTooLarge(): avoid extra blank …
jimklimov 337cc4e
CpsScriptTest: methodTooLargeExceptionRealistic(): expect updated wor…
jimklimov 1ea2ffe
Merge branch 'master' into logging-exMTL
jimklimov 7fd3dd2
Merge branch 'master' into logging-exMTL
jimklimov 1692243
Merge branch 'master' into logging-exMTL
jimklimov 92b74f1
Merge branch 'master' into logging-exMTL
jimklimov 3cb7ea5
Merge branch 'master' into logging-exMTL
jimklimov 095ff23
Merge remote-tracking branch 'upstream/master' as of 2025-10-04 into …
jimklimov ede9176
CpsFlowExecution, CpsScriptTest: mvn spotless:apply
jimklimov 2766a10
CpsFlowExecution: rephrase Jenkins Shared Library/JSL (doc book term)…
jimklimov ef62003
CpsFlowExecution:: simplify end-of-line regex
jimklimov 0612903
Merge branch 'master' into logging-exMTL
jimklimov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
Uh oh!
There was an error while loading. Please reload this page.