Violation plugin iterates over files not mentioned in report

[vzabawski]

Violation plugin scans all files in pull request and tries to get information about them even if ther are not mentioned in report file:

Here's my plugin configuration:

This behavior slows down build process because build is not finished until all files are processed.

Even after I changed commentOnlyChangedContent to true, nothing changed and plugin keeps iteration over all files in pull request.

Info

• Plugin version: 1.97
• Jenkins version: 2.150.2

[tomasbjerre]

First requests gets all changes.

The following requests are looking for old comments. Old comments made by findings in previous analysis. With a previous report file where it they might have been mentioned.

https://github.com/tomasbjerre/violation-comments-to-bitbucket-server-lib/blob/master/src/main/java/se/bjurr/violations/comments/bitbucketserver/lib/BitbucketServerCommentsProvider.java#L111

[vzabawski]

Is there any way to change such behavior? Right now, those API calls take 90% of build time for some jobs.

I've tried to adjust plugin parameters, so current parameters look like:

Those adjustments allowed to decrease build time by ~30% but API calls part still takes a lot of time. So, is there any way to change this behavior? Thanks in advance.

[tomasbjerre]

If the createSingleFileComments is false, then I think it should be no problem avoiding these calls.

[vzabawski]

It can be achieved by enabling "Create one big comment with all violations" in UI, right?

[tomasbjerre]

Yes. This plugin just needs a change in the implementation first.

[tomasbjerre]

Just released a fix in version 1.99 now.