[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16883472#comment-16883472 ] ASF subversion and git services commented on LUCENE-8907: - Commit 6d79cc9e443b957baf72abcc7757c5edfb7ff1c1 in lucene-solr's branch refs/heads/jira/SOLR-13565 from Tomoko Uchida [ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=6d79cc9 ] LUCENE-8907: Move change logs for LUCENE-8778 and following issues to the 9.0.0 updates section. > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Assignee: Tomoko Uchida >Priority: Blocker > Fix For: 8.2 > > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16883094#comment-16883094 ] Tomoko Uchida commented on LUCENE-8907: --- I opened to carry on the discussion here: LUCENE-8911. > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Assignee: Tomoko Uchida >Priority: Blocker > Fix For: 8.2 > > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16883043#comment-16883043 ] ASF subversion and git services commented on LUCENE-8907: - Commit 6d79cc9e443b957baf72abcc7757c5edfb7ff1c1 in lucene-solr's branch refs/heads/master from Tomoko Uchida [ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=6d79cc9 ] LUCENE-8907: Move change logs for LUCENE-8778 and following issues to the 9.0.0 updates section. > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Assignee: Tomoko Uchida >Priority: Blocker > Fix For: 8.2 > > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16883027#comment-16883027 ] ASF subversion and git services commented on LUCENE-8907: - Commit 824a196d780a5d5694a579969c2aaf45554555c8 in lucene-solr's branch refs/heads/branch_8_2 from Tomoko Uchida [ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=824a196d ] LUCENE-8907: Revert LUCENE-8778 and succeeding commits. > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Assignee: Tomoko Uchida >Priority: Blocker > Fix For: 8.2 > > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16883016#comment-16883016 ] ASF subversion and git services commented on LUCENE-8907: - Commit 59c7eb92cfcc766ee6866fe14ec609b09d41fcf6 in lucene-solr's branch refs/heads/branch_8x from Tomoko Uchida [ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=59c7eb9 ] LUCENE-8907: Revert LUCENE-8778 and succeeding commits. > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Assignee: Tomoko Uchida >Priority: Blocker > Fix For: 8.2 > > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.14#76016) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16882545#comment-16882545 ] Tomoko Uchida commented on LUCENE-8907: --- I don't want to stop 8.2 release for this issue. If there is no strong objection within 24 hours, let me revert the changes (mentioned in my previous comment) for now and open another issue to discuss about backporting LUCENE-8778 to 8.x. It would be easy to cherry-pick those changes to the 8x branch again. > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Assignee: Tomoko Uchida >Priority: Blocker > Fix For: 8.2 > > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[
https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16882500#comment-16882500
]
Tomoko Uchida commented on LUCENE-8907:
---
Hi,
I picked the commits which should be reverted from commit log mail. There are
five commits: all is mine and there is no conflicts other than
lucene/CHANGES.txt on the reverted 8x branch. All tests is OK and that passed
precommit.
{code}
git revert 2804d00137957dc27dce922103c6286bbc7a9eca
git revert 7e05bd717309088d3b35cba6be0f70fd14b3c8d0
git revert d1678a3a68685acd1432dfadcd5f49fa817273ad
git revert 12e3451fb80e151b16f51493761b9dfc580f8fa0
git revert 0cc1753e76eb61000c1d3513448719e5f7923e48
{code}
I think we can delay the discussion about backporting LUCENE-8778 into 8.x with
transparent backwards compatibility. (From my perspective, the backporting to
the 8x branch was my fault and we do not need to change the SPI loader until
release 9.0.)
> Provide backward compatibility for loading analysis factories
> -
>
> Key: LUCENE-8907
> URL: https://issues.apache.org/jira/browse/LUCENE-8907
> Project: Lucene - Core
> Issue Type: Task
> Components: modules/analysis
>Reporter: Tomoko Uchida
>Assignee: Tomoko Uchida
>Priority: Blocker
> Fix For: 8.2
>
>
> The changes in LUCENE-8778 have breaking changes in the analysis factory
> interface and custom factories implemented by users / 3rd parties will be
> affected. We need to keep some backwards compatibility during 8.x.
> Please see the discussion in SOLR-13593 for details.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16882483#comment-16882483 ] Jan Høydahl commented on LUCENE-8907: - Reverting sounds to cause least friction, not delaying 8.2. Unless a patch is presented within say 24h, which is very simple and straight forward.? > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Assignee: Tomoko Uchida >Priority: Blocker > Fix For: 8.2 > > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16882309#comment-16882309 ] Uwe Schindler commented on LUCENE-8907: --- I agree partly with Adrien. But we can still allow to use old factories in 8.2, in a lenient way: If the factory has no NAME field, it generates one using old algorithm - and ideally would log a warning. The biggest problem ist that we can't log warnings, which is not good. Solr could do this. Sorry for coming up with this issue so late, but I did not notice that the whole change was committed to 8.x. I have here a customer project with many custom analyzers, so it came to my mind. > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Assignee: Tomoko Uchida >Priority: Blocker > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16882074#comment-16882074 ] Adrien Grand commented on LUCENE-8907: -- I have a slight preference for reverting on 8.x and have this change only in 9.0. My worry is that the backward compatibility layer would either need to introduce leniency (factories without a NAME) or stronger checks (Same NAME and class name), and it could end up causing as much trouble as the original change. [~thetaphi] What do you think? > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Assignee: Tomoko Uchida >Priority: Blocker > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16881999#comment-16881999 ] Tomoko Uchida commented on LUCENE-8907: --- Hi, reverting is a feasible option because there have been no changes - as far as I know - that depends on LUCENE-8778 other than my commits. I am sure that I can safely revert all my related commits so far on branch_8x (though I need to confirm that). Personally I would prefer reverting instead of blocking release 8.2, which seems to happen soon. Maybe we can rethink the way to safely include the changes into 8.x again a little later. Are there any other suggestions/opinions? > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Priority: Blocker > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16881982#comment-16881982 ] Adrien Grand commented on LUCENE-8907: -- I don't like 2, it feels too breaking to me for a minor release. I like 1 better but then I think we should also fail if an analysis factory has a NAME constant that is not the same as the name that would be derived from the class name? Another option could be to revert from 8.x. In any case we should add migration instructions to lucene/MIGRATE.txt on master. bq. some warning messages would be helpful We never log from Lucene since this is a library. > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Priority: Blocker > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
[jira] [Commented] (LUCENE-8907) Provide backward compatibility for loading analysis factories
[ https://issues.apache.org/jira/browse/LUCENE-8907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16881944#comment-16881944 ] Tomoko Uchida commented on LUCENE-8907: --- There may be two options: 1. Completely emulate the factory loading method of Lucene 8.1. (i.e., copy the old logic and use it if the "NAME" field does not exist in a factory.) 2. Ignore factories which have no proper "NAME" field. (We have to note the behaviour somewhere.) Either way, some warning messages would be helpful to let users know that they have to change their code by Lucene 9.0 (can we show such "warnings"?). > Provide backward compatibility for loading analysis factories > - > > Key: LUCENE-8907 > URL: https://issues.apache.org/jira/browse/LUCENE-8907 > Project: Lucene - Core > Issue Type: Task > Components: modules/analysis >Reporter: Tomoko Uchida >Priority: Blocker > > The changes in LUCENE-8778 have breaking changes in the analysis factory > interface and custom factories implemented by users / 3rd parties will be > affected. We need to keep some backwards compatibility during 8.x. > Please see the discussion in SOLR-13593 for details. -- This message was sent by Atlassian JIRA (v7.6.3#76005) - To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
