[GitHub] [solr] renatoh commented on pull request #1873: Removing List.toArray calls with pre-sized Arrays

2023-09-15 Thread via GitHub
renatoh commented on PR #1873: URL: https://github.com/apache/solr/pull/1873#issuecomment-1721339817 @epugh Hey Eric, could you please give me a hand with this PR? I do not understand what the issue is. Thanks -- This is an automated message from the Apache Git Service. To respond

[GitHub] [solr] renatoh commented on pull request #1873: Removing List.toArray calls with pre-sized Arrays

2023-09-07 Thread via GitHub
renatoh commented on PR #1873: URL: https://github.com/apache/solr/pull/1873#issuecomment-1710188125 @epugh is it possible that you have a look at tell me what's wrong with my commit? I don't what the issue is -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [solr] renatoh commented on pull request #1873: Removing List.toArray calls with pre-sized Arrays

2023-09-06 Thread via GitHub
renatoh commented on PR #1873: URL: https://github.com/apache/solr/pull/1873#issuecomment-1707871257 @cpoerschke I did another merge of origin/main, it looks like the checks have now passed, or haven't they? -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [solr] renatoh commented on pull request #1873: Removing List.toArray calls with pre-sized Arrays

2023-09-04 Thread via GitHub
renatoh commented on PR #1873: URL: https://github.com/apache/solr/pull/1873#issuecomment-1705604995 @cpoerschke do you know why patches are applied e.g. "Applying: SOLR-16871: Synchronize on a larger block to avoid race condition in CoordinatorHttpSolrCall init" as a part of the build?

[GitHub] [solr] renatoh commented on pull request #1873: Removing List.toArray calls with pre-sized Arrays

2023-09-04 Thread via GitHub
renatoh commented on PR #1873: URL: https://github.com/apache/solr/pull/1873#issuecomment-1705440090 @cpoerschke I did that, and it shows me that my fork is up to date, but I am getting: (#error: patch failed: solr/core/src/java/org/apache/solr/cli/AssertTool.java:35 -- This is an aut

[GitHub] [solr] renatoh commented on pull request #1873: Removing List.toArray calls with pre-sized Arrays

2023-08-31 Thread via GitHub
renatoh commented on PR #1873: URL: https://github.com/apache/solr/pull/1873#issuecomment-1702234144 hmm, now JavaSpotless check is failing on a class I have not touched, this one: src/java/org/apache/solr/cloud/api/collections/OverseerCollectionMessageHandler.java -- This is an auto

[GitHub] [solr] renatoh commented on pull request #1873: Removing List.toArray calls with pre-sized Arrays

2023-08-30 Thread via GitHub
renatoh commented on PR #1873: URL: https://github.com/apache/solr/pull/1873#issuecomment-1700433722 i don't see whats why the Precommit is complaining about SpellCheckComponent.java, I did run tidy again, and no changes appear. -- This is an automated message from the Apache Git Service.

[GitHub] [solr] renatoh commented on pull request #1873: Removing List.toArray calls with pre-sized Arrays

2023-08-30 Thread via GitHub
renatoh commented on PR #1873: URL: https://github.com/apache/solr/pull/1873#issuecomment-1699695874 @madrob I might have been wrong with the static code analysis, looks like it is even a compiler warning: ![Screenshot 2023-08-30 at 21 04 30](https://github.com/apache/solr/assets/1034413

[GitHub] [solr] renatoh commented on pull request #1873: Removing List.toArray calls with pre-sized Arrays

2023-08-30 Thread via GitHub
renatoh commented on PR #1873: URL: https://github.com/apache/solr/pull/1873#issuecomment-1699681345 @madrob The one build in to intellij, not sure if this can be added to the build, but I assume this would be picked up by any other static code analyzer too. ![Screenshot 2023-08-30 at