bamaer opened a new issue, #8774:
URL: https://github.com/apache/hop/issues/8774

   ### Apache Hop version?
   
   2.20.0-SNAPSHOT
   
   ### Java version?
   
   21
   
   ### Operating system
   
   macOS
   
   ### What happened?
   
   ### Apache Hop version?
   
   2.20.0-SNAPSHOT (main)
   
   ### Java version?
   
   21
   
   ### Operating system
   
   macOS
   
   ### What happened?
   
   After switching to another project in Hop GUI, Tools → Lint → Show Lint 
Results lists the findings of the previously enabled project together with 
those of the current one.
   
   ### Cause
   
   - `LintResultsManager` keeps results per file path for the whole GUI 
session, independent of the project.
   - On `HopGuiProjectAfterEnabled`, `ProjectLoadedLintExtension` starts 
`BackgroundLintService.lintProjectAsync`. It merges results file by file 
through `updateResultsForFile`, which only replaces the entries of the files it 
lints. Entries for files of the previous project are never removed.
   - `LintResultsManager.clearResults()` is only called from 
`LinterConfigPlugin.applyEnabledState()`, when the linter is disabled.
   - `LintResultsPanel` shows `getAllResults()` when no file or folder filter 
is set, so both projects' findings are listed.
   - Tools → Lint → Lint Project... replaces all results through 
`updateResults()`, which hides the problem until the next project switch.
   
   ### Related
   
   - A background project lint that is still running for the previous project 
keeps publishing its results after the switch. `lintProjectAsync` has no 
generation or cancel check, unlike `lintGraph`.
   - `LintCheckTracker` lives as long as `BackgroundLintService` and skips 
files that have not changed since they were checked. Clearing the results on a 
project switch without resetting the tracker leaves a project that is switched 
back to without findings.
   
   ### Steps to reproduce
   
   1. Enable project A, which has lint findings, and wait for the background 
lint to finish.
   2. Switch to project B.
   3. Open Tools → Lint → Show Lint Results.
   
   ### Expected
   
   Only the findings of project B are listed.
   
   ### Issue Priority
   
   Priority: 3
   
   ### Issue Component
   
   Component: Hop Gui
   
   ### Issue Priority
   
   Priority: 2
   
   ### Issue Component
   
   Component: Hop Gui


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to