I did a bit of poking around, and it’s not super clear. The Flink project tried and ended up abandoning it, https://issues.apache.org/jira/browse/FLINK-4370 <https://issues.apache.org/jira/browse/FLINK-4370>, though they did adopt a check style using IntelliJ. Apache Kyuubi went the way of README that walks you through setup of a number of aspects of IntelligenceJ: https://kyuubi.readthedocs.io/en/latest/develop_tools/idea_setup.html <https://kyuubi.readthedocs.io/en/latest/develop_tools/idea_setup.html>
I’d like to use the same inspection profile as the community uses, so that the red-ness tells me when I need to do something ;-). And if it isn’t red, then it’s okay, we are okay with it as a community. So…. As an idea, how about we start with David Smiley’s inspection profile….? I’ll see if I can figure out how to make that file ship in gradle/ide/intellij-idea/ directory, and have our gradle idea task copy that file to the ./idea/inspectionProfiles directory. I don’t know if I can make it be the default, maybe if I name it “Project_Default.xml”. That will get us all to looking at the same set of warnings, at least for those using IntelliJ. Then, as we evolve our opinions on this, we can just update that file. Thoughts? > On Aug 24, 2022, at 6:06 PM, David Smiley <dsmi...@apache.org> wrote: > > Do other projects do that? FWIW I tailor an IDE-wide inspection profile to > my liking, tweaking what's enabled and also the severity level to reduce > red-ness when I don't think it's warranted. > > ~ David Smiley > Apache Lucene/Solr Search Developer > http://www.linkedin.com/in/davidwsmiley > <http://www.linkedin.com/in/davidwsmiley> > > > On Tue, Aug 23, 2022 at 12:34 PM Eric Pugh <ep...@opensourceconnections.com > <mailto:ep...@opensourceconnections.com>> > wrote: > >> I was thinking about how to make this process less painful, and maybe how >> to capture what we as a community think are warnings that we actually care >> about. >> >> IntelliJ appears to have “Inspection Profiles” >> https://www.jetbrains.com/help/idea/code-inspection.html#access-inspections-and-settings >> >> <https://www.jetbrains.com/help/idea/code-inspection.html#access-inspections-and-settings> >> < >> https://www.jetbrains.com/help/idea/code-inspection.html#access-inspections-and-settings >> >> <https://www.jetbrains.com/help/idea/code-inspection.html#access-inspections-and-settings>> >> that we can customize. >> >> It appears that there is a .idea/InspectionProfiles directory that maybe >> our gradle build could somehow tie into…. Run gradlew idea and have our >> inspection file setup for us? >> >> >> >> >>> On Aug 18, 2022, at 12:03 AM, Houston Putman <hous...@apache.org> wrote: >>> >>> I completely understand your concern and frustration for the >>> solrj-zookeeper PR David, it's been in the works for a while, and these >>> refactorings are very hard on you and the contributor you are working >> with. >>> >>> Eric and I talked, and given that there are very few giant PRs that are >> in >>> progress, we agreed this work can likely continue after the >> solrj-zookeeper >>> PR is merged. >>> >>> Going forward though, I think the strategy is going to be to attack >>> package-by-package (addressing all issues in that package), instead of >>> intelliJ issue by intelliJ issue (addressing all packages at the same >>> time). That way, the PRs are confined to a specific area, so for example >>> the solrj-zookeeper work would be unaffected unless Eric decided to >> tackle >>> the solrj packages, which he could obviously postpone. >>> >>> I think we can all agree that the Solr codebase can be a mess, and things >>> are not always done the way that they should be. Projects need >> maintenance >>> in order to survive. I think it's great that Eric is putting in >> tremendous >>> time and effort in order to improve the project in a pretty thankless >> way. >>> >>> Anyways happy to entertain further discussion on the topic, but I think >> the >>> new strategy is the best compromise of moving forward without making >> things >>> hard for others trying to contribute. >>> >>> - Houston >>> >>> On Sat, Jul 30, 2022 at 4:38 PM Ishan Chattopadhyaya < >>> ichattopadhy...@gmail.com> wrote: >>> >>>>> I ask that you please suspend sweeping changes because they are a real >>>> impediment to other branches/PRs that move lots of files. This is >>>> happening right now: https://github.com/apache/solr/pull/943 (for >> SolrJ) >>>> >>>> +1, I don't feel the value such changes bring to the project outweigh >> the >>>> disruption. >>>> >>>> On Fri, Jul 29, 2022 at 9:26 PM Christine Poerschke (BLOOMBERG/ LONDON) >> < >>>> cpoersc...@bloomberg.net> wrote: >>>> >>>>> PR 916 was a subset of PR 912 and I've just resolved its conflicts. >>>>> >>>>> Also https://issues.apache.org/jira/browse/SOLR-16318 started, in case >>>>> that helps. >>>>> >>>>> Christine >>>>> >>>>> From: dev@solr.apache.org At: 07/26/22 15:10:03 UTC+1:00To: >>>>> dev@solr.apache.org >>>>> Subject: Re: Cleaning up IntelliJ warnings in code base >>>>> >>>>> Not sure about https://github.com/apache/solr/pull/916 but >>>>> https://github.com/apache/solr/pull/912 conflicts in a ton of places >> and >>>>> haven't looked at updating it. >>>>> >>>>> Kevin Risden >>>>> >>>>> >>>>> On Tue, Jul 26, 2022 at 9:51 AM Eric Pugh < >>>> ep...@opensourceconnections.com >>>>>> >>>>> wrote: >>>>> >>>>>> Hi all…. Are there any updates to the two PR’s related to WIP about >>>>>> updating to JDK 11 features? >>>>>> >>>>>> Once I’m done with https://issues.apache.org/jira/browse/SOLR-16300 < >>>>>> https://issues.apache.org/jira/browse/SOLR-16300> Migrate from >>>>> deprecated >>>>>> assertThat() to org.hamcrest.MatcherAssert.assertThat(), it looks like >>>>> MANY >>>>>> of the remaining things to do all are related to upgrading to more >>>> modern >>>>>> JDK language features…. >>>>>> >>>>>> I’m happy to do that, but don’t want to step on the work that is >>>>> progress? >>>>>> >>>>>> >>>>>> Eric >>>>>> >>>>>>> On Jul 4, 2022, at 2:23 PM, Eric Pugh < >>>> ep...@opensourceconnections.com >>>>>> >>>>>> wrote: >>>>>>> >>>>>>> I wanted to share a quick update! >>>>>>> >>>>>>> First off though, thanks Christine for reviewing many of the PR’s of >>>>>> been pushing up. As I’ve been editing the files I’ve found myself >>>>>> thinking…. WWCD in this situation[1]? >>>>>>> >>>>>>> Here are the PR’s that are getting close, and I’m just looking for >>>> some >>>>>> LGTM’s: >>>>>>> >>>>>>> * SOLR-16281: review boxing unboxing < >>>>>> https://github.com/apache/solr/pull/930> >>>>>>> * SOLR-16280: simplify for loop < >>>>> https://github.com/apache/solr/pull/929 >>>>>>> >>>>>>> * SOLR-16278: remove unused declarations < >>>>>> https://github.com/apache/solr/pull/927> >>>>>>> * SOLR-16223: unused throws identified by IntelliJ (round 2) < >>>>>> https://github.com/apache/solr/pull/926>. <— I learned how to >> use >>>>>> IntelliJ better after doing the first round of redundant throws review >>>>> ;-) >>>>>>> * SOLR-16275 use standard charset instead of string forName version < >>>>>> https://github.com/apache/solr/pull/925> >>>>>>> * SOLR-16276: redundant variable in test < >>>>>> https://github.com/apache/solr/pull/924> >>>>>>> >>>>>>> >>>>>>> Also, as I am moving forward, I saw two PR’s that I don’t quite >>>>>> understand, and I don’t want to step on their toes: >>>>>>> >>>>>>> experimental sub-set 1 of "WIP - Use up to JDK 11 features” < >>>>>> https://github.com/apache/solr/pull/916> and WIP - Use up to JDK 11 >>>>>> features <https://github.com/apache/solr/pull/912>, these each both >>>> look >>>>>> to move us up to modern Java standards across all the code…. I’m >>>>>> wondering though if some of the work I’m doing is going to collide, >>>> like >>>>>> SOLR-16281? Should we prioritize one of these efforts to get to >> done >>>>>> done? There appears to be a TON of great stuff in these updates to >>>>> JDK11 >>>>>> features, so would love to see them get in sooner versus later, and >>>> then >>>>> I >>>>>> can go back to cleaning up what remains…. < >>>>>> https://github.com/apache/solr/pull/916#partial-pull-merging> >>>>>>> >>>>>>> ERic >>>>>>> >>>>>>> <https://github.com/apache/solr/pull/912#partial-pull-merging> < >>>>>> https://github.com/apache/solr/pull/912#partial-pull-merging> >>>>>>> >>>>>>> >>>>>>> >>>>>>> [1] WWCD: What Would Christine Do? >>>>>>> >>>>>>>> On Jun 27, 2022, at 5:41 PM, Eric Pugh < >>>>> ep...@opensourceconnections.com >>>>>> <mailto:ep...@opensourceconnections.com>> wrote: >>>>>>>> >>>>>>>> I pushed up some more PR’s for warnings from IntelliJ. >>>>>>>> >>>>>>>> Here are the ones that if folks agree, are ready for merging. >>>>>>>> >>>>>>>> * SOLR-162680: toString is not required when crafting a string >>>>> message < >>>>>> https://github.com/apache/solr/pull/922> >>>>>>>> * SOLR-16224: class level variables that can be safely made local, >>>> and >>>>>> are boilerpl… <https://github.com/apache/solr/pull/921> >>>>>>>> * SOLR-16269 clean up use of final keyword < >>>>>> https://github.com/apache/solr/pull/919> >>>>>>>> * SOLR-16270: address dangling javadocs < >>>>>> https://github.com/apache/solr/pull/918> >>>>>>>> >>>>>>>> Another one that I am looking at is to eliminate all of the import >>>>>> java.util.*; type imports, which I believe is what we have in the >> main >>>>>> code base. I wanted to make sure that is a pattern to follow in the >>>> unit >>>>>> tests too? >>>>>>>> >>>>>>>> SOLR-16271: clean up imports (WIP) < >>>>>> https://github.com/apache/solr/pull/920> >>>>>>>> >>>>>>>> Eric >>>>>>>> >>>>>>>> <https://github.com/apache/solr/pull/918#partial-pull-merging> < >>>>>> https://github.com/apache/solr/pull/918#partial-pull-merging> >>>>>>>> <https://github.com/apache/solr/pull/919#partial-pull-merging> < >>>>>> https://github.com/apache/solr/pull/919#partial-pull-merging> >>>>>>>> <https://github.com/apache/solr/pull/921#partial-pull-merging> < >>>>>> https://github.com/apache/solr/pull/921#partial-pull-merging> >>>>>>>> <https://github.com/apache/solr/pull/922#partial-pull-merging> < >>>>>> https://github.com/apache/solr/pull/922#partial-pull-merging> >>>>>>>> >>>>>>>>> On Jun 27, 2022, at 11:25 AM, Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>> >>>>>>>>> Thanks David for the comments in the ticket, and it makes sense to >>>>>> treat stylistic warnings different from deprecations. >>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>>>>> On Jun 24, 2022, at 10:11 PM, David Smiley <dsmi...@apache.org >>>>>> <mailto:dsmi...@apache.org>> wrote: >>>>>>>>>> >>>>>>>>>> How to handle certain deprecations is IMO a different matter than >>>>>> stylistic-ish warnings. I'll respond in SOLR-16263. >>>>>>>>>> >>>>>>>>>> ~ David Smiley >>>>>>>>>> Apache Lucene/Solr Search Developer >>>>>>>>>> http://www.linkedin.com/in/davidwsmiley < >>>>>> http://www.linkedin.com/in/davidwsmiley> >>>>>>>>>> >>>>>>>>>> On Fri, Jun 24, 2022 at 12:19 PM Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>> I created https://issues.apache.org/jira/browse/SOLR-16263 < >>>>>> https://issues.apache.org/jira/browse/SOLR-16263>, "Migrate tests >> from >>>>>> deprecate http clients to http2 equivalents”. I wanted to confirm >>>> that >>>>>> we ARE actually deprecating and removing HttpSolrClient in favour of >>>>>> Http2SolrClient ??? >>>>>>>>>> >>>>>>>>>> Since it was deprecated in 9.0, what does that mean for updating >>>> the >>>>>> tests? They can only be updated to Http2SolrClient in the main >>>> branch? >>>>>> Or can the tests at least be updated to Http2SolrClient in main and >>>>>> branch_9x? >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>>> On Jun 24, 2022, at 10:10 AM, Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>>> >>>>>>>>>>> I wanted to send a heads up that I plan on merging >>>>>> https://github.com/apache/solr/pull/900 < >>>>>> https://github.com/apache/solr/pull/900> on Monday as I’ll be back >>>>>> online, starting the long drive back home to Virginia. >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>>> On Jun 13, 2022, at 5:10 PM, Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>>>> >>>>>>>>>>>> Made it to Kansas ;-). Tomorrow I’ll have an hour before I >>>> sign >>>>>> off for two weeks of vacation time in Colorado.... Is it too risky >> to >>>>>> merge this to main? I was thinking that keeping this patch up to >> date >>>>>> would be very difficult with ongoing changes, but now rethinking >> that…. >>>>>>>>>>>> >>>>>>>>>>>> Eric >>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>>> On Jun 13, 2022, at 9:50 AM, Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>>>>> >>>>>>>>>>>>> Thanks Christine for putting in some suggestions! I’m passing >>>>>> through Saline MO (https://goo.gl/maps/cZGCTZy5aN8UHNo8A < >>>>>> >>>>> >>>>> >>>> >> https://www.google.com/maps/place/Saline,+MI+48176/@42.1765171,-83.7954921,14z/d >>>>> >>>>> >>>> >> ata=!3m1!4b1!4m5!3m4!1s0x883cbacfff5b8a61:0xb76c9793c1fef3cd!8m2!3d42.1667072!4d >>>>> -83.7816075 >>>>> < >>>> >> https://www.google.com/maps/place/Saline,+MI+48176/@42.1765171,-83.7954921,14z/data=!3m1!4b1!4m5!3m4!1s0x883cbacfff5b8a61:0xb76c9793c1fef3cd!8m2!3d42.1667072!4d-83.7816075 >>>>> >>>>>> ) >>>>>> and just finished my pass through all the solr/core/src/test-files. >>>> I’m >>>>>> checking out the suggestions you made. >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>>> On Jun 12, 2022, at 4:44 PM, Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>>>>>> >>>>>>>>>>>>>> While sitting in the car from Virginia to Missouri I have been >>>>>> fixing up typos and grammar. >>>>>>>>>>>>>> >>>>>>>>>>>>>> https://github.com/apache/solr/pull/900 < >>>>>> https://github.com/apache/solr/pull/900> >>>>>>>>>>>>>> Tomorrow on the way to Kansas I will finish up the last few >>>>> tests. >>>>>>>>>>>>>> >>>>>>>>>>>>>> I would love to get some review.... my thought is if any of >>>>> the >>>>>> changes I made don’t seem right I will back them out. Erik Hatcher >>>> gave >>>>> me >>>>>> one example, the use of the word collocated versus colated. >>>>>>>>>>>>>> >>>>>>>>>>>>>> I would love to merge this on Tuesday on my way to Colorado. >>>>>>>>>>>>>> >>>>>>>>>>>>>> On Thu, Jun 9, 2022 at 4:40 PM Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>>>>>> Thanks Mike for fixing my backport issue…… >>>>>>>>>>>>>> >>>>>>>>>>>>>> So, I’ve wended my way through 84 files focusing on grammar >>>> and >>>>>> typos: https://github.com/apache/solr/pull/900 < >>>>>> https://github.com/apache/solr/pull/900> >>>>>>>>>>>>>> >>>>>>>>>>>>>> One thing I wanted to highlight, the >>>>>> AffinityPlacementFactory.java < >>>>>> >>>>> >>>>> >>>> >> https://github.com/apache/solr/pull/900/files#diff-660c049913f52a982b864a41bc795 >>>>> b5803c9f72bfa17ef1efcf72eb9e2354b5f >>>>> < >>>> >> https://github.com/apache/solr/pull/900/files#diff-660c049913f52a982b864a41bc795b5803c9f72bfa17ef1efcf72eb9e2354b5f >>>>> >>>>>> >>>>>> has a typo in the Exception message. I fixed it, but wondered if >>>> there >>>>>> was potentially some small chance that the original spelling is >>>> important >>>>>> for some reason? >>>>>>>>>>>>>> >>>>>>>>>>>>>> I’d love a thumbs up that I’m going in the right direction, >>>> I’ll >>>>>> probably pick this up again on Saturday…. >>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> Eric >>>>>>>>>>>>>> >>>>>>>>>>>>>>> On Jun 3, 2022, at 4:52 PM, Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Okay, I merged the other day…. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Firstly, in terms of a workflow, I looked at the >>>>>> Solr-NightlyTests-main Jenkins test, and in fact the one that ran >> after >>>>> my >>>>>> commit has two tests that failed that look like normal build >> failures…. >>>>>> Versus what I committed: >>>>>> >>>>> >>>>> >>>> >> https://ci-builds.apache.org/job/Solr/job/Solr-NightlyTests-main/445/#showFailur >>>>> esLink >>>>>> < >>>>>> >>>>> >>>>> >>>> >> https://ci-builds.apache.org/job/Solr/job/Solr-NightlyTests-main/445/#showFailur >>>>> esLink >>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Are there any other Jenkin’s builds I should keep an eye on? >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Secondly, what is the Jenkins Test Failure Report that I >>>> should >>>>>> keep an eye on? I’ve seen >>>>>> http://fucit.org/solr-jenkins-reports/failure-report.html < >>>>>> http://fucit.org/solr-jenkins-reports/failure-report.html> in the >>>> past, >>>>>> and is this the one we want? It looks like this report aggregates >>>> from >>>>>> MANY different Jenkins jobs, so I would be looking for a failure that >>>>>> starts up post my commit? Or, do you recommend that I focus on >>>>>> >>>>> >>>> >> http://fucit.org/solr-jenkins-reports/history-trend-of-recent-failures.html >>>>>> < >>>>>> >>>>> >>>> >> http://fucit.org/solr-jenkins-reports/history-trend-of-recent-failures.html >>>>>> >>>>>> ??/ >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Lastly, give it say 5 days and then back port to branch_9x??? >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Thanks! >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Eric >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> On Jun 1, 2022, at 8:11 AM, Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> One last question…. “Squash and Merge” right? We don’t >>>> care >>>>>> about all my interim commits…. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> On May 31, 2022, at 6:37 PM, Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> Cool. >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> So here is an example of what I’ll merge in a day or so: >>>>>> https://github.com/apache/solr/pull/885 < >>>>>> https://github.com/apache/solr/pull/885> >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> And I’m going to start on a new PR for >>>>>> https://issues.apache.org/jira/browse/SOLR-16224 < >>>>>> https://issues.apache.org/jira/browse/SOLR-16224> that is about >>>> looking >>>>>> at how fields are defined in the tests. >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> On May 31, 2022, at 12:41 PM, Houston Putman < >>>>>> hous...@apache.org <mailto:hous...@apache.org>> wrote: >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> What about back porting, would you want these back ported >>>> to >>>>>> 8 and 9? Or just 9? >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> I would say just main and branch_9x >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> On Tue, May 31, 2022 at 12:39 PM Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>>>>>>>>>> Thanks for the response Mike… >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> So I finished up going through the test code, and yeah, >>>>>> wow…. Doing it one file at a time was educational at least ;-). >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>> >>>>> >>>>> >>>> >> https://github.com/apache/solr/compare/main...epugh:intellij_inspired_cleanups?e >>>>> xpand=1 >>>>> < >>>> >> https://github.com/apache/solr/compare/main...epugh:intellij_inspired_cleanups?expand=1 >>>>> >>>>>> < >>>>>> >>>>> >>>>> >>>> >> https://github.com/apache/solr/compare/main...epugh:intellij_inspired_cleanups?e >>>>> xpand=1 >>>>> < >>>> >> https://github.com/apache/solr/compare/main...epugh:intellij_inspired_cleanups?expand=1 >>>>> >>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> In terms of a workflow, should I open up a single JIRA and >>>>>> then list under it a task for each type of fix? And then merge each >>>>>> individual type of fix? >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> So a single JIRA issue “Examine IntelliJ Warnings in Test >>>>>> Code”, and then a JIRA under that for each type, starting with “Remove >>>>>> Exceptions not thrown by Method”? Then merge each one to main, wait >> a >>>>> few >>>>>> days to make sure no spike in errors, and then do the next one? >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> What about back porting, would you want these back ported >>>> to >>>>>> 8 and 9? Or just 9? >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> Eric >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> On May 27, 2022, at 8:50 PM, Mike Drob <md...@mdrob.com >>>>>> <mailto:md...@mdrob.com>> wrote: >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Declaring an unused thrown exception in tests isn't the >>>>> most >>>>>> critical change, but cleaning this up might help us discover >> accidental >>>>> API >>>>>> signature changes in the future. If a test throws an exception then >>>> JUnit >>>>>> will figure it out and fail the test anyway, which is probably what we >>>>> want >>>>>> to do regardless. >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> I'm +0 on this, like I am not going to go out of my way >>>> to >>>>>> refactor that, but now that you've done it I don't want to just throw >>>>> away >>>>>> your work so it's probably fine to commit. I hope this was some >>>> automated >>>>>> fix you could apply instead of doing manually. >>>>>>>>>>>>>>>>>>> But I'm also not going to review it, so I hope you trust >>>>> the >>>>>> automated tooling and are willing to volunteer watching Jenkins for a >>>> few >>>>>> days after. :) >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Unused exceptions anywhere under src/main I would be >>>> _very_ >>>>>> interested in, on the other hand. >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Mike >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> On Fri, May 27, 2022 at 7:33 PM Eric Pugh < >>>>>> ep...@opensourceconnections.com <mailto: >>>> ep...@opensourceconnections.com >>>>>>> >>>>>> wrote: >>>>>>>>>>>>>>>>>>> So, going through and cleaning up unused throwing of >>>>>> exceptions, I’ve touched all these files listed below. I was >>>> thinking I >>>>>> would do ONE commit for all of the “remove unused Exception”…. >>>> Before I >>>>>> keep going, wanted to make sure that makes sense….. >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/analysis/CommonGramsPhraseQueryTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/analysis/PathHierarchyTokenizerFactoryTest.ja >>>>> va >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>> solr/core/src/test/org/apache/solr/analysis/TestLuceneMatchVersion.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/analysis/TestReversedWildcardFilterFactory.ja >>>>> va >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/analysis/TokenizerChainTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/ActionThrottleTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/AssignBackwardCompatibilityTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>> solr/core/src/test/org/apache/solr/cloud/ChaosMonkeyShardSplitTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/CollectionPropsTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/CollectionsAPISolrJTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/ConcurrentCreateRoutedAliasTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/ConfigSetApiLockingTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/CreateRoutedAliasTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/DeleteShardTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/DistribJoinFromCollectionTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/ForceLeaderTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/HttpPartitionTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/LeaderElectionTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/OverseerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/ReindexCollectionTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/SSLMigrationTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/SolrCLIZkUtilsTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >> solr/core/src/test/org/apache/solr/cloud/TestAuthenticationFramework.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/TestBaseStatsCacheCloud.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/TestCloudDeleteByQuery.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/TestCloudInspectUtil.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/TestCloudPivotFacet.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/TestHashPartitioner.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/TestPrepRecovery.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/TestRebalanceLeaders.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/TestSSLRandomization.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/TestStressCloudBlindAtomicUpdates.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/TestTlogReplica.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/cloud/ZkCLITest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/api/collections/AsyncCallRequestStatusR >>>>> esponseTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/api/collections/BackupRestoreApiErrorCo >>>>> nditionsTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/api/collections/CollectionApiLockingTes >>>>> t.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/api/collections/CollectionTooManyReplic >>>>> asTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/api/collections/ReplicaPropertiesBase.j >>>>> ava >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/api/collections/TestLocalFSCloudBackupR >>>>> estore.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cloud/api/collections/TestReplicaProperties.j >>>>> ava >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/cluster/events/impl/CollectionsRepairEventLis >>>>> tenerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/AlternateDirectoryTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/core/ConfigureRecoveryStrategyTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/DirectoryFactoryTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/HelloStream.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/ResourceLoaderTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/SOLR749Test.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/SolrCoreTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>> solr/core/src/test/org/apache/solr/core/TestBackupRepositoryFactory.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/TestCodecSupport.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/TestConfig.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/TestConfigSetImmutable.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/TestConfigSetProperties.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/TestConfigSets.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/TestShardHandlerFactory.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/core/TestXIncludeConfig.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/core/snapshots/TestSolrCloudSnapshots.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/core/snapshots/TestSolrCoreSnapshots.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/DocumentAnalysisRequestHandlerTest.ja >>>>> va >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/FieldAnalysisRequestHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>> solr/core/src/test/org/apache/solr/handler/MoreLikeThisHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/handler/PingRequestHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/handler/SearchHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/handler/TestContainerPlugin.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >> solr/core/src/test/org/apache/solr/handler/TestIncrementalCoreBackup.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/handler/TestSnapshotCoreBackup.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/TestSolrConfigHandlerCloud.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/handler/V2ApiIntegrationTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>> solr/core/src/test/org/apache/solr/handler/V2ClusterAPIMappingTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/handler/V2UpdateAPIMappingTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/XmlUpdateRequestHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/CoreAdminOperationTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/CoreMergeIndexesAdminHandlerTes >>>>> t.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/HealthCheckHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/IndexSizeEstimatorTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/LukeRequestHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>> solr/core/src/test/org/apache/solr/handler/admin/MetricsHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/ShowFileRequestHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/handler/admin/SplitHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >> solr/core/src/test/org/apache/solr/handler/admin/StatsReloadRaceTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/SystemInfoHandlerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/handler/admin/TestApiFramework.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>> solr/core/src/test/org/apache/solr/handler/admin/TestCollectionAPIs.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/V2CollectionBackupsAPIMappingTe >>>>> st.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/V2CollectionsAPIMappingTest.jav >>>>> a >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/api/V2CollectionAPIMappingTest. >>>>> java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/api/V2CoreAPIMappingTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/api/V2NodeAPIMappingTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/admin/api/V2ShardsAPIMappingTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/BadComponentTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/CustomHighlightComponentTes >>>>> t.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/DebugComponentTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/DistributedExpandComponentT >>>>> est.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/DistributedFacetExistsSmall >>>>> Test.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/DistributedSpellCheckCompon >>>>> entTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/DistributedSuggestComponent >>>>> Test.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/FacetPivotSmallTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/ShardRequestTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/SpellCheckComponentTest.jav >>>>> a >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/SuggestComponentContextFilt >>>>> erQueryTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/SuggestComponentTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/TestExpandComponent.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/TestHttpShardHandlerFactory >>>>> .java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/component/TestPivotHelperCode.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>> solr/core/src/test/org/apache/solr/handler/loader/JavabinLoaderTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/tagger/EmbeddedSolrNoSerializeTest.ja >>>>> va >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/handler/tagger/XmlInterpolationTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/highlight/DummyHighlighter.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/highlight/HighlighterMaxOffsetTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/highlight/HighlighterTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/internal/csv/CharBufferTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/legacy/TestLegacyField.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/legacy/TestLegacyNumericUtils.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/metrics/JvmMetricsTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/metrics/MetricsConfigTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >> solr/core/src/test/org/apache/solr/metrics/SolrCoreMetricManagerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/metrics/SolrMetricManagerTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/metrics/reporters/SolrJmxReporterTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/metrics/reporters/SolrSlf4jReporterTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/request/TestFaceting.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/response/TestJavabinTupleStreamParser.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/response/TestRawResponseWriter.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/response/TestRetrieveFieldsOptimizer.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/response/TestSolrQueryResponse.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/response/transform/TestChildDocTransformer.ja >>>>> va >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/response/transform/TestChildDocTransformerHie >>>>> rarchy.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/response/transform/TestExplainDocTransformer. >>>>> java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/response/transform/TestSubQueryTransformer.ja >>>>> va >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/response/transform/TestSubQueryTransformerCro >>>>> ssCore.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/rest/TestManagedResource.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/rest/TestRestManager.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/rest/schema/TestDynamicFieldCollectionResourc >>>>> e.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/rest/schema/TestDynamicFieldResource.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/rest/schema/TestFieldCollectionResource.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/rest/schema/TestFieldResource.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/rest/schema/TestFieldTypeCollectionResource.j >>>>> ava >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >> solr/core/src/test/org/apache/solr/rest/schema/TestFieldTypeResource.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/rest/schema/TestSchemaNameResource.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/rest/schema/TestSchemaResource.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/rest/schema/TestSchemaSimilarityResource.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/rest/schema/TestSchemaVersionResource.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/rest/schema/TestSerializedLuceneMatchVersion. >>>>> java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/rest/schema/TestUniqueKeyFieldResource.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/CurrencyFieldTypeTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/DocValuesMultiTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/EnumFieldTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>> solr/core/src/test/org/apache/solr/schema/ExternalFileFieldSortTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/IndexSchemaTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>>> >>>>> >>>> >> solr/core/src/test/org/apache/solr/schema/ManagedSchemaRoundRobinCloudTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/PolyFieldTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/PreAnalyzedFieldTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/RankFieldTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/SchemaWatcherTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/SpatialRPTFieldTypeTest.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/TestCloudManagedSchema.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/TestManagedSchema.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> solr/core/src/test/org/apache/solr/schema/TestSortableTextField.java >>>>>>>>>>>>>>>>>>> modified: >>>>>> >>>> solr/core/src/test/org/apache/solr/schema/TestUseDocValuesAsStored.java >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> On May 27, 2022, at 1:52 PM, David Smiley < >>>>>> dsmi...@apache.org <mailto:dsmi...@apache.org>> wrote: >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> IntelliJ is produced by a company and I have no idea how >>>>>> they go about selecting what the default inspections (what IntelliJ >>>> calls >>>>>> these) are. Maybe it was one person there, maybe it was arbitrary by >>>>>> whoever wrote the inspection, or maybe they had some more thoughtful >>>>>> approach that looked at literature. Regardless, I disagree with some >>>> of >>>>>> their choices. I think we should base our decisions on what >>>> inspections >>>>> to >>>>>> address for ourselves, not just because JetBrains included them. I >>>>>> routinely adjust my IntelliJ inspection settings to not harass me >> about >>>>>> some matters that I consider to be frivolous. For example boolean >>>>>> expression simplifications -- where we as a project (when a part of >>>>> Lucene) >>>>>> have chosen "== false" to be clearer than an exclamation point >> adjacent >>>>> to >>>>>> a boolean expression. >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> If we do some of this: Agreed on picking exactly one >>>>>> "inspection" and scoping to just one module at first. Could increase >>>> to >>>>>> more commits in the same PR if you get good feedback. >>>>>>>>>>>>>>>>>>>> Personally, I wouldn't do this endeavor unless the >>>>>> particular inspection is something that particularly motivates me / >>>> was a >>>>>> pet-peeve. >>>>>>>>>>>>>>>>>>>> I think "getting to green" is a toal lost cause unless >>>> we >>>>>> were to enforce a particular configured list of inspections (which is >>>>>> IntelliJ only, remember). >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> ~ David Smiley >>>>>>>>>>>>>>>>>>>> Apache Lucene/Solr Search Developer >>>>>>>>>>>>>>>>>>>> http://www.linkedin.com/in/davidwsmiley < >>>>>> http://www.linkedin.com/in/davidwsmiley> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> On Fri, May 27, 2022 at 12:52 PM Shawn Heisey < >>>>>> apa...@elyograg.org <mailto:apa...@elyograg.org>> wrote: >>>>>>>>>>>>>>>>>>>> On 5/27/2022 8:24 AM, Eric Pugh wrote: >>>>>>>>>>>>>>>>>>>>> Hey all, was poking around at a unit test while >>>> watching >>>>>> TV and >>>>>>>>>>>>>>>>>>>>> noticed lots of warnings from IntelliJ, little stuff >>>>> like >>>>>> exceptions >>>>>>>>>>>>>>>>>>>>> being thrown that don’t need to be thrown, unused >>>>>> variables, or typos. >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> In eclipse, there are THOUSANDS of warnings. And last I >>>>>> checked, even a >>>>>>>>>>>>>>>>>>>> bunch of errors. But I was able to build >>>> 10.0.0-SNAPSHOT >>>>>> successfully. >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> Thanks, >>>>>>>>>>>>>>>>>>>> Shawn >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> >>>>>> --------------------------------------------------------------------- >>>>>>>>>>>>>>>>>>>> To unsubscribe, e-mail: dev-unsubscr...@solr.apache.org >>>>>> <mailto:dev-unsubscr...@solr.apache.org> >>>>>>>>>>>>>>>>>>>> For additional commands, e-mail: >>>> dev-h...@solr.apache.org >>>>>> <mailto:dev-h...@solr.apache.org> >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> _______________________ >>>>>>>>>>>>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>>> 434.466.1467 | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>>>>>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>>>>>>>>>>>> This e-mail and all contents, including attachments, is >>>>>> considered to be Company Confidential unless explicitly stated >>>> otherwise, >>>>>> regardless of whether attachments are marked as such. >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> _______________________ >>>>>>>>>>>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>>> 434.466.1467 | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>>>>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>>>>>>>>>>> This e-mail and all contents, including attachments, is >>>>>> considered to be Company Confidential unless explicitly stated >>>> otherwise, >>>>>> regardless of whether attachments are marked as such. >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> _______________________ >>>>>>>>>>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>>> 434.466.1467 | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>>>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>>>>>>>>>> This e-mail and all contents, including attachments, is >>>>>> considered to be Company Confidential unless explicitly stated >>>> otherwise, >>>>>> regardless of whether attachments are marked as such. >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> _______________________ >>>>>>>>>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>>> 434.466.1467 | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>>>>>>>>> This e-mail and all contents, including attachments, is >>>>>> considered to be Company Confidential unless explicitly stated >>>> otherwise, >>>>>> regardless of whether attachments are marked as such. >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> _______________________ >>>>>>>>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>>> 434.466.1467 | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>>>>>>>> This e-mail and all contents, including attachments, is >>>>>> considered to be Company Confidential unless explicitly stated >>>> otherwise, >>>>>> regardless of whether attachments are marked as such. >>>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>>>> _______________________ >>>>>>>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>>> 434.466.1467 | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>>>>>>> This e-mail and all contents, including attachments, is >>>>>> considered to be Company Confidential unless explicitly stated >>>> otherwise, >>>>>> regardless of whether attachments are marked as such. >>>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> _______________________ >>>>>>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>>> 434.466.1467 | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>>>>>> This e-mail and all contents, including attachments, is >>>>> considered >>>>>> to be Company Confidential unless explicitly stated otherwise, >>>> regardless >>>>>> of whether attachments are marked as such. >>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> _______________________ >>>>>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>>> 434.466.1467 | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>>>>> This e-mail and all contents, including attachments, is >>>> considered >>>>>> to be Company Confidential unless explicitly stated otherwise, >>>> regardless >>>>>> of whether attachments are marked as such. >>>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> _______________________ >>>>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>>> 434.466.1467 | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>>>> This e-mail and all contents, including attachments, is >>>> considered >>>>>> to be Company Confidential unless explicitly stated otherwise, >>>> regardless >>>>>> of whether attachments are marked as such. >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>> _______________________ >>>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>>> 434.466.1467 | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>>> This e-mail and all contents, including attachments, is considered >>>>> to >>>>>> be Company Confidential unless explicitly stated otherwise, regardless >>>> of >>>>>> whether attachments are marked as such. >>>>>>>>>> >>>>>>>>> >>>>>>>>> _______________________ >>>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>>> 434.466.1467 >>>>>> | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>>> This e-mail and all contents, including attachments, is considered >>>> to >>>>>> be Company Confidential unless explicitly stated otherwise, regardless >>>> of >>>>>> whether attachments are marked as such. >>>>>>>>> >>>>>>>> >>>>>>>> _______________________ >>>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>> 434.466.1467 >>>>>> | http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>>> This e-mail and all contents, including attachments, is considered >>>> to >>>>>> be Company Confidential unless explicitly stated otherwise, regardless >>>> of >>>>>> whether attachments are marked as such. >>>>>>>> >>>>>>> >>>>>>> _______________________ >>>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | >>>> 434.466.1467 >>>>> | >>>>>> http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>>> This e-mail and all contents, including attachments, is considered to >>>>> be >>>>>> Company Confidential unless explicitly stated otherwise, regardless of >>>>>> whether attachments are marked as such. >>>>>>> >>>>>> >>>>>> _______________________ >>>>>> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | 434.466.1467 >>>> | >>>>>> http://www.opensourceconnections.com < >>>>>> http://www.opensourceconnections.com/> | My Free/Busy < >>>>>> http://tinyurl.com/eric-cal> >>>>>> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >>>>>> >>>>> >>>>> >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterpri >>>>> se-search-server-third-edition-raw >>>>> < >>>> >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >>>>> >>>>>> >>>>>> >>>>>> This e-mail and all contents, including attachments, is considered to >>>> be >>>>>> Company Confidential unless explicitly stated otherwise, regardless of >>>>>> whether attachments are marked as such. >>>>>> >>>>>> >>>>> >>>>> >>>>> >>>> >> >> _______________________ >> Eric Pugh | Founder & CEO | OpenSource Connections, LLC | 434.466.1467 | >> http://www.opensourceconnections.com < >> http://www.opensourceconnections.com/ >> <http://www.opensourceconnections.com/>> | My Free/Busy < >> http://tinyurl.com/eric-cal <http://tinyurl.com/eric-cal>> >> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed < >> https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw >> >> <https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw>> >> >> This e-mail and all contents, including attachments, is considered to be >> Company Confidential unless explicitly stated otherwise, regardless of >> whether attachments are marked as such. _______________________ Eric Pugh | Founder & CEO | OpenSource Connections, LLC | 434.466.1467 | http://www.opensourceconnections.com <http://www.opensourceconnections.com/> | My Free/Busy <http://tinyurl.com/eric-cal> Co-Author: Apache Solr Enterprise Search Server, 3rd Ed <https://www.packtpub.com/big-data-and-business-intelligence/apache-solr-enterprise-search-server-third-edition-raw> This e-mail and all contents, including attachments, is considered to be Company Confidential unless explicitly stated otherwise, regardless of whether attachments are marked as such.
