You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-dev@jackrabbit.apache.org by Chetan Mehrotra <ch...@gmail.com> on 2015/12/10 12:36:53 UTC
Remove/Disable ordered property indexes in trunk
Given that ordered property indexes have been deprecated long time
back it would be better if we remove the corresponding code or at
least disable the OSGi components for it
I would prefer removal of code as otherwise we need to take care of
that code also if any cross cutting refactoring is to be performed
(say some change which touch all indexes)
Thoughts?
Chetan Mehrotra
Re: Remove/Disable ordered property indexes in trunk
Posted by Chetan Mehrotra <ch...@gmail.com>.
On Thu, Dec 10, 2015 at 7:43 PM, Davide Giannella <da...@apache.org> wrote:
> Can any of you please file an issue and assign it to myself?
Done with OAK-3768
Chetan Mehrotra
Re: Remove/Disable ordered property indexes in trunk
Posted by Davide Giannella <da...@apache.org>.
On 10/12/2015 12:37, Alex Parvulescu wrote:
> I'm also in favour or removing, but maybe do it as a two phase approach,
> first drop the code but leave a dummy component that will log a warning in
> case the specific index type is used, next on phase 2 drop the warning.
>
+1 for this.
Can any of you please file an issue and assign it to myself?
Phase 1 for Oak 1.4, phase 2 for Oak 1.6.
Cheers
Davide
Re: Remove/Disable ordered property indexes in trunk
Posted by Alex Parvulescu <al...@gmail.com>.
I'm also in favour or removing, but maybe do it as a two phase approach,
first drop the code but leave a dummy component that will log a warning in
case the specific index type is used, next on phase 2 drop the warning.
On Thu, Dec 10, 2015 at 12:36 PM, Chetan Mehrotra <chetan.mehrotra@gmail.com
> wrote:
> Given that ordered property indexes have been deprecated long time
> back it would be better if we remove the corresponding code or at
> least disable the OSGi components for it
>
> I would prefer removal of code as otherwise we need to take care of
> that code also if any cross cutting refactoring is to be performed
> (say some change which touch all indexes)
>
> Thoughts?
>
> Chetan Mehrotra
>
Re: Remove/Disable ordered property indexes in trunk
Posted by Francesco Mari <ma...@gmail.com>.
Indexing is not really my area of expertise, but I'm all in favour of
removing long-time deprecated code.
2015-12-10 12:36 GMT+01:00 Chetan Mehrotra <ch...@gmail.com>:
> Given that ordered property indexes have been deprecated long time
> back it would be better if we remove the corresponding code or at
> least disable the OSGi components for it
>
> I would prefer removal of code as otherwise we need to take care of
> that code also if any cross cutting refactoring is to be performed
> (say some change which touch all indexes)
>
> Thoughts?
>
> Chetan Mehrotra
>