github, generate report

Report generation failed on phase "make_report",
step "Generate report".
Link: https://github.com/checkstyle/checkstyle/actions/runs/9920624278

github, generate report

EDIT: Wrong report

https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/83d5013_2024155550/reports/diff/index.html

github, generate report

https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/83d5013_2024172535/reports/diff/index.html

@mahfouz72 There is a file conflict.

github, generate report

Report generation failed on phase "make_report",
step "Generate report".
Link: https://github.com/checkstyle/checkstyle/actions/runs/9924090074

github, generate report

Report generation failed on phase "make_report",
step "Generate report".
Link: https://github.com/checkstyle/checkstyle/actions/runs/9928212690

github, generate report

I surprised on out of memory error for ParenPad , I thought it is light Check. If you find reason of this problem please share it.

Something similar happened before in another PR and @rnveach said here it could be having too many violations that's why we ran into memory. That is the only reason I can think of now. Note that the option is space for the current config.
but I still think this is not a good amount of violations won't be that big to cause heap space error.

base

       <module name="ParenPad">
           <property name="option" value="space"/>
           <property name="tokens" value=""/>
       </module>

patch

      <module name="ParenPad">
           <property name="option" value="space"/>     
           <property name="tokens" value="RECORD_PATTERN_DEF"/>
       </module>

<property name="tokens" value=""/> lol this empty string means all not none ? If yes then OOM is expected we have a violation for all tokens to have space after parens. This will be a huge number of violations

github, generate report

Report generation failed on phase "make_report",
step "Generate report".
Link: https://github.com/checkstyle/checkstyle/actions/runs/9928997699

https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/17788b1_2024200613/reports/diff/index.html

I surprised on out of memory error for ParenPad , I thought it is light Check. If you find reason of this problem please share it.

This in one of the checks I see OOM in most frequently due to nesting of ASTs in some test inputs and massive number of violations

• no space: https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/83d5013_2024172535/reports/diff/index.html
• space: https://checkstyle-diff-reports.s3.us-east-2.amazonaws.com/83d5013_2024172535/reports/diff/index.html

@rnveach @nrmancuso ping