[GitHub] [solr] gerlowskija commented on pull request #1210: SOLR-16347: Skip JerseyApp creation when v2 disabled

[GitBox <gi...@apache.org>]

gerlowskija commented on PR #1210:
URL: https://github.com/apache/solr/pull/1210#issuecomment-1380917707

   > There is not reason to disable V2 in the first place
   
   Noble, you were the one that added this sysprop back in 71abe1?
   
   And as far as I can tell - most of the reasons a user might've wanted this back then are still valid today:
   
   - the v2 code is still evolving and might have avoidable bugs that can be closed off by disabling.
   - users configuring security might not want the extra surface area to consider
   - those worried about performance might not want to pay the cost of bootstrapping multiple API surfaces at startup
   
   > V2 is not an experimental feature anymore
   
   You're right that it's been around forever, but it's absolutely still an experimental feature as evidenced by [this mailing list discussion](https://lists.apache.org/thread/t342hl7lvt5b4qmb5vrrh7tvmdjlbb22), which resulted in [this JIRA ticket](https://issues.apache.org/jira/browse/SOLR-15780) and the related [note in our ref-guide](https://solr.apache.org/guide/solr/latest/configuration-guide/v2-api.html).
   
   ----
   
   If you want to remove the flag, I'd be happy to have that discussion.  But while its there and is documented in the ref-guide, it makes sense for it to fully work.


-- 
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@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org