This breaks branch_8x for me - Java 8 doesn’t recognise the “since” attribute 
in @Deprecated.

> On 2 Jul 2020, at 13:10, is...@apache.org wrote:
> 
> This is an automated email from the ASF dual-hosted git repository.
> 
> ishan pushed a commit to branch branch_8x
> in repository https://gitbox.apache.org/repos/asf/lucene-solr.git
> 
> 
> The following commit(s) were added to refs/heads/branch_8x by this push:
>     new 755d31c  SOLR-14066: Deprecate DIH
> 755d31c is described below
> 
> commit 755d31c4933a5b9bb8e39e5e9fb50b17934ff336
> Author: Ishan Chattopadhyaya <is...@apache.org>
> AuthorDate: Thu Jul 2 16:38:31 2020 +0530
> 
>    SOLR-14066: Deprecate DIH
> ---
> solr/CHANGES.txt                                                 | 7 +++++--
> solr/contrib/dataimporthandler-extras/src/java/overview.html     | 2 +-
> solr/contrib/dataimporthandler/README.txt                        | 9 +++++++++
> .../org/apache/solr/handler/dataimport/DataImportHandler.java    | 3 +++
> solr/contrib/dataimporthandler/src/java/overview.html            | 2 +-
> solr/example/example-DIH/README.txt                              | 2 ++
> solr/solr-ref-guide/src/dataimport-screen.adoc                   | 2 ++
> solr/solr-ref-guide/src/solr-upgrade-notes.adoc                  | 3 +++
> ...-structured-data-store-data-with-the-data-import-handler.adoc | 2 ++
> solr/webapp/web/css/angular/dataimport.css                       | 3 ++-
> solr/webapp/web/partials/dataimport.html                         | 1 +
> 11 files changed, 31 insertions(+), 5 deletions(-)
> 
> diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
> index 59d1972..a0bdcd4 100644
> --- a/solr/CHANGES.txt
> +++ b/solr/CHANGES.txt
> @@ -351,6 +351,9 @@ Other Changes
> 
> * SOLR-14022: Deprecate CDCR (Joel Bernstein, Ishan Chattopadhyaya)
> 
> +* SOLR-14066: Data Import Handler is deprecated. It is scheduled to be 
> removed as of 9.0 and a community supported
> +  package for the same may now be used instead. (Ishan Chattopadhyaya, 
> janhoy)
> +
> ==================  8.5.2 ==================
> 
> Consult the LUCENE_CHANGES.txt file for additional, low level, changes in 
> this release.
> @@ -6941,8 +6944,8 @@ Other Changes
>   (David Smiley)
> 
> * SOLR-8842: security rules made more foolproof by asking the requesthandler  
> about the well known
> -  permission name.
>  The APIs are also modified to ue 'index' as the unique identifier instead of 
> name.
> -  Name is an optional attribute
>  now and only to be used when specifying well-known permissions (noble)
> +  permission name.  The APIs are also modified to ue 'index' as the unique 
> identifier instead of name.
> +  Name is an optional attribute  now and only to be used when specifying 
> well-known permissions (noble)
> 
> * SOLR-5616: Simplifies grouping code to use ResponseBuilder.needDocList() to 
> determine if it needs to
>   generate a doc list for grouped results. (Steven Bower, Keith Laban, Dennis 
> Gove)
> diff --git a/solr/contrib/dataimporthandler-extras/src/java/overview.html 
> b/solr/contrib/dataimporthandler-extras/src/java/overview.html
> index a60a25e..5a55432 100644
> --- a/solr/contrib/dataimporthandler-extras/src/java/overview.html
> +++ b/solr/contrib/dataimporthandler-extras/src/java/overview.html
> @@ -16,6 +16,6 @@
> -->
> <html>
> <body>
> -Apache Solr Search Server: DataImportHandler Extras contrib
> +Apache Solr Search Server: DataImportHandler Extras contrib. <b>This contrib 
> module is deprecated as of 8.6</b>
> </body>
> </html>
> diff --git a/solr/contrib/dataimporthandler/README.txt 
> b/solr/contrib/dataimporthandler/README.txt
> index c969872..1d78b93 100644
> --- a/solr/contrib/dataimporthandler/README.txt
> +++ b/solr/contrib/dataimporthandler/README.txt
> @@ -14,3 +14,12 @@ running Solr you set the following system properties:
>   -Duser.language=xx -Duser.country=YY -Duser.timezone=ZZZ
> 
> where xx, YY, and ZZZ are consistent with any database server's configuration.
> +
> +Deprecation notice
> +------------------
> +This contrib module is deprecated as of v8.6, scheduled for removal in Solr 
> 9.0.
> +The reason is that DIH is no longer being maintained in a manner we feel is 
> necessary in order to keep it
> +healthy and secure. Also it was not designed to work with SolrCloud and does 
> not meet current performance requirements.
> +
> +The project hopes that the community will take over maintenance of DIH as a 
> 3rd party package (See SOLR-14066 for more details). Please reach out to us 
> at the dev@ mailing list if you want to help.
> +
> diff --git 
> a/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/DataImportHandler.java
>  
> b/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/DataImportHandler.java
> index 9fb8b04..a1fbcc2 100644
> --- 
> a/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/DataImportHandler.java
> +++ 
> b/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/DataImportHandler.java
> @@ -63,8 +63,10 @@ import static 
> org.apache.solr.handler.dataimport.DataImporter.IMPORT_CMD;
>  * <p>
>  * <b>This API is experimental and subject to change</b>
>  *
> + * @deprecated since 8.6
>  * @since solr 1.3
>  */
> +@Deprecated(since = "8.6")
> public class DataImportHandler extends RequestHandlerBase implements
>         SolrCoreAware {
> 
> @@ -95,6 +97,7 @@ public class DataImportHandler extends RequestHandlerBase 
> implements
>     Map<String,String> macro = new HashMap<>();
>     macro.put("expandMacros", "false");
>     defaults = SolrParams.wrapDefaults(defaults, new MapSolrParams(macro));
> +    log.warn("Data Import Handler is deprecated as of Solr 8.6. See 
> SOLR-14066 for more details.");
>   }
> 
>   @Override
> diff --git a/solr/contrib/dataimporthandler/src/java/overview.html 
> b/solr/contrib/dataimporthandler/src/java/overview.html
> index 4bb2c8b..4c2d595 100644
> --- a/solr/contrib/dataimporthandler/src/java/overview.html
> +++ b/solr/contrib/dataimporthandler/src/java/overview.html
> @@ -16,6 +16,6 @@
> -->
> <html>
> <body>
> -Apache Solr Search Server: DataImportHandler contrib
> +Apache Solr Search Server: DataImportHandler contrib. <b>This contrib module 
> is deprecated as of 8.6.</b>
> </body>
> </html>
> diff --git a/solr/example/example-DIH/README.txt 
> b/solr/example/example-DIH/README.txt
> index 30c37eb..0123c4f 100644
> --- a/solr/example/example-DIH/README.txt
> +++ b/solr/example/example-DIH/README.txt
> @@ -16,6 +16,8 @@
> Solr DataImportHandler example configuration
> --------------------------------------------
> 
> +NOTE: The DataImportHandler is deprecated as of v8.6. See SOLR-14066 for 
> more details.
> +
> To run this multi-core example, use the "-e" option of the bin/solr script:
> 
>> bin/solr -e dih
> diff --git a/solr/solr-ref-guide/src/dataimport-screen.adoc 
> b/solr/solr-ref-guide/src/dataimport-screen.adoc
> index 647814c..1f28cd5 100644
> --- a/solr/solr-ref-guide/src/dataimport-screen.adoc
> +++ b/solr/solr-ref-guide/src/dataimport-screen.adoc
> @@ -16,6 +16,8 @@
> // specific language governing permissions and limitations
> // under the License.
> 
> +WARNING: The Data Import Handler is deprecated as of v8.6 and is scheduled 
> to be removed in 9.0.
> +
> The Dataimport screen shows the configuration of the DataImportHandler (DIH) 
> and allows you start, and monitor the status of, import commands as defined 
> by the options selected on the screen and defined in the configuration file.
> 
> .The Dataimport Screen
> diff --git a/solr/solr-ref-guide/src/solr-upgrade-notes.adoc 
> b/solr/solr-ref-guide/src/solr-upgrade-notes.adoc
> index b3c1ce5..1f1ec97 100644
> --- a/solr/solr-ref-guide/src/solr-upgrade-notes.adoc
> +++ b/solr/solr-ref-guide/src/solr-upgrade-notes.adoc
> @@ -84,6 +84,9 @@ For more information about how to use this, see the section 
> <<exporting-result-s
> *Deprecations* 
> 
> * Cross Data Center Replication (CDCR), in its current form, is deprecated 
> and is scheduled to be removed in 9.0. Please refer to SOLR-14022.
> +*Deprecations*
> +
> +* Data Import Handler is deprecated and is scheduled to be removed in 9.0. A 
> community supported version of this may be available as an external package 
> to be used with the package management system. For more details, please refer 
> to SOLR-14066.
> 
> === Solr 8.5
> 
> diff --git 
> a/solr/solr-ref-guide/src/uploading-structured-data-store-data-with-the-data-import-handler.adoc
>  
> b/solr/solr-ref-guide/src/uploading-structured-data-store-data-with-the-data-import-handler.adoc
> index d9f6cf8..5d925ff 100644
> --- 
> a/solr/solr-ref-guide/src/uploading-structured-data-store-data-with-the-data-import-handler.adoc
> +++ 
> b/solr/solr-ref-guide/src/uploading-structured-data-store-data-with-the-data-import-handler.adoc
> @@ -17,6 +17,8 @@
> // specific language governing permissions and limitations
> // under the License.
> 
> +WARNING: The Data Import Handler is deprecated as of v8.6 and is scheduled 
> to be removed in 9.0.
> +
> Many search applications store the content to be indexed in a structured data 
> store, such as a relational database. The Data Import Handler (DIH) provides 
> a mechanism for importing content from a data store and indexing it.
> 
> In addition to relational databases, DIH can index content from HTTP based 
> data sources such as RSS and ATOM feeds, e-mail repositories, and structured 
> XML where an XPath processor is used to generate fields.
> diff --git a/solr/webapp/web/css/angular/dataimport.css 
> b/solr/webapp/web/css/angular/dataimport.css
> index 6afb096..ad37896 100644
> --- a/solr/webapp/web/css/angular/dataimport.css
> +++ b/solr/webapp/web/css/angular/dataimport.css
> @@ -258,7 +258,8 @@ limitations under the License.
>   background-image: url( ../../img/ico/information.png );
> }
> 
> -#content #dataimport #error
> +#content #dataimport #error,
> +#content #dataimport #deprecation_message
> {
>   background-color: #f00;
>   background-image: url( ../../img/ico/construction.png );
> diff --git a/solr/webapp/web/partials/dataimport.html 
> b/solr/webapp/web/partials/dataimport.html
> index dd11f54..a27be07 100644
> --- a/solr/webapp/web/partials/dataimport.html
> +++ b/solr/webapp/web/partials/dataimport.html
> @@ -18,6 +18,7 @@ limitations under the License.
> 
>   <div ng-show="!hasHandlers">The solrconfig.xml file for this index does not 
> have an operational DataImportHandler defined!</div>
>   <div id="frame" ng-show="hasHandlers">
> +    <div id="deprecation_message">The Data Import Handler is deprecated as 
> of Solr 8.6 and may be removed in a future release. A community supported 
> package for may be used instead (See SOLR-14066 for details).</div>
> 
>     <div id="error" ng-show="error"></div>
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to