Skip to content
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

[core] Add DEBUG log when applying incremental analysis #608

Closed
blindpirate opened this issue Sep 9, 2017 · 2 comments

Comments

@blindpirate
Copy link

commented Sep 9, 2017

It seems that no debug message is outputted when running incremental analysis. When I run pmd 5.8.1 with incremental analysis on and log level debug, I always receive message Processing file xxxx in every pmd analysis process. It's outputted here . There's no debug information at all when cache hit or miss here.

From log, I can't learn whether the incremental analysis is done or not.

@jsotuyod jsotuyod self-assigned this Sep 9, 2017
@jsotuyod jsotuyod added this to the 6.0.0 milestone Sep 9, 2017
jsotuyod added a commit to Monits/pmd that referenced this issue Sep 10, 2017
 - Resolves pmd#608
 - Log succesful cache loading
 - Log hits / misses on debug, reason for miss too
 - Invite users to enable incremental analysis
 - Provide a new section on the web for the feature. May be promoted to
a top level page at some point.
@jsotuyod jsotuyod added the has:pr label Sep 10, 2017
jsotuyod added a commit to Monits/pmd that referenced this issue Sep 10, 2017
 - Resolves pmd#608
 - Log succesful cache loading
 - Log hits / misses on debug, reason for miss too
 - Invite users to enable incremental analysis
 - Provide a new section on the web for the feature. May be promoted to
a top level page at some point.
@jsotuyod

This comment has been minimized.

Copy link
Member

commented Sep 10, 2017

@blindpirate PR #614 has been submitted to address this among other concerns.

Let me know if for some reason you still think it's not enough / we are missing visibility.

@blindpirate

This comment has been minimized.

Copy link
Author

commented Sep 10, 2017

Thanks!

@jsotuyod jsotuyod changed the title Add DEBUG log when applying incremental analysis [core] Add DEBUG log when applying incremental analysis Sep 10, 2017
@adangel adangel closed this in #614 Sep 29, 2017
blindpirate added a commit to gradle/gradle that referenced this issue Oct 9, 2017
…age for pmd incremental analysis (#2961)"

Since there're some issues unreleased in pmd plugin:

pmd/pmd#618
pmd/pmd#608

This reverts commit:

79e74d9
4c51b74
d98129f
c543f98
60fa200
bf27585
f7a4bf8
blindpirate added a commit to gradle/gradle that referenced this issue Oct 10, 2017
…age for pmd incremental analysis (#2961)"

Since there're some issues unreleased in pmd plugin:

pmd/pmd#618
pmd/pmd#608

This reverts commit:

79e74d9
4c51b74
d98129f
c543f98
60fa200
bf27585
f7a4bf8
blindpirate added a commit to gradle/gradle that referenced this issue Oct 10, 2017
…age for pmd incremental analysis (#2961)"

Since there're some issues unreleased in pmd plugin:

pmd/pmd#618
pmd/pmd#608

This reverts commit:

79e74d9
4c51b74
d98129f
c543f98
60fa200
bf27585
f7a4bf8
blindpirate added a commit to gradle/gradle that referenced this issue Oct 10, 2017
…age for pmd incremental analysis (#2961)" (#3125)

Since there're some issues unreleased in pmd plugin:

pmd/pmd#618
pmd/pmd#608

This reverts commit:

79e74d9
4c51b74
d98129f
c543f98
60fa200
bf27585
f7a4bf8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.