cpoerschke commented on pull request #202:
URL: https://github.com/apache/solr/pull/202#issuecomment-929215631
> So I'll just wait on the other issue to get resolved then? Any TO-DOs
before then?
No TO-DOs that I can think of, thanks for the ping! Once the other issue is
resolved
cpoerschke commented on pull request #202:
URL: https://github.com/apache/solr/pull/202#issuecomment-929215631
> So I'll just wait on the other issue to get resolved then? Any TO-DOs
before then?
No TO-DOs that I can think of, thanks for the ping! Once the other issue is
resolved
cpoerschke commented on pull request #202:
URL: https://github.com/apache/solr/pull/202#issuecomment-913644039
> Hmm, running the full test suite locally via `./gradlew test` gives two
reproducible failures: ...
This is, I think, because the `@JsonProperty` annotations don't yet
cpoerschke commented on pull request #202:
URL: https://github.com/apache/solr/pull/202#issuecomment-912684782
Hmm, running the full test suite locally via `./gradlew test` gives two
reproducible failures:
```
gradlew :solr:core:test --tests
cpoerschke commented on pull request #202:
URL: https://github.com/apache/solr/pull/202#issuecomment-911854652
Thanks @bitnahian for completing the test TO-DO items!
I just added two commits to the branch:
* @gus-asf's suggestion to deprecate the Integer method variant
* a
cpoerschke commented on pull request #202:
URL: https://github.com/apache/solr/pull/202#issuecomment-898673372
> Attempted to run tests under
org/apache/solr/update/processor/TimeRoutedAliasUpdateProcessorTest.java but
tests are currently ignored.
Looks like
cpoerschke commented on pull request #202:
URL: https://github.com/apache/solr/pull/202#issuecomment-897714718
Thanks @bitnahian for opening this pull request!
I'm not really familiar with this area of the code but via `git grep`
noticed that