You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "rmuir (via GitHub)" <gi...@apache.org> on 2025/10/08 20:44:19 UTC
[I] ci: change error-prone check to be more explicit rather than based on $CI env variable [lucene]
rmuir opened a new issue, #15311:
URL: https://github.com/apache/lucene/issues/15311
### Description
In the CI system, we want to suffer the pain of error-prone just for the "checks without tests" job.
When running the tests, we would prefer that compilation is fast (other job is using error-prone). For example on windows runner, significant portion of "tests" job is actually compilation:
```
Summary of top-20 task execution times (aggregated by unique name, possibly running in parallel):
1344.83 sec. test
553.56 sec. compileTestJava
233.73 sec. compileJava
```
It would be better to just pass something like a `-Perrorprone` in the "checks without tests" job, and don't pass it in the "tests" job?
--
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: issues-unsubscribe@lucene.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [I] ci: change error-prone check to be more explicit rather than based on $CI env variable [lucene]
Posted by "rmuir (via GitHub)" <gi...@apache.org>.
rmuir commented on issue #15311:
URL: https://github.com/apache/lucene/issues/15311#issuecomment-3383238205
I made a stab on #15312
--
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: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [I] ci: change error-prone check to be more explicit rather than based on $CI env variable [lucene]
Posted by "rmuir (via GitHub)" <gi...@apache.org>.
rmuir closed issue #15311: ci: change error-prone check to be more explicit rather than based on $CI env variable
URL: https://github.com/apache/lucene/issues/15311
--
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: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [I] ci: change error-prone check to be more explicit rather than based on $CI env variable [lucene]
Posted by "dweiss (via GitHub)" <gi...@apache.org>.
dweiss commented on issue #15311:
URL: https://github.com/apache/lucene/issues/15311#issuecomment-3383185323
The option is there already -
```
-Pvalidation.errorprone=false
```
will turn off errorprone.
--
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: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [I] ci: change error-prone check to be more explicit rather than based on $CI env variable [lucene]
Posted by "rmuir (via GitHub)" <gi...@apache.org>.
rmuir commented on issue #15311:
URL: https://github.com/apache/lucene/issues/15311#issuecomment-3383193912
Will make a PR and let's see if it makes a dent. Thank you
--
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: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [I] ci: change error-prone check to be more explicit rather than based on $CI env variable [lucene]
Posted by "dweiss (via GitHub)" <gi...@apache.org>.
dweiss commented on issue #15311:
URL: https://github.com/apache/lucene/issues/15311#issuecomment-3383186378
try ```./gradlew allOptions```.
--
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: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org