Message5380
On 08/10/15 02:12, anatoly techtonik wrote:
> On Fri, Oct 2, 2015 at 3:39 PM, John Kristensen <john@jerrykan.com
> <mailto:john@jerrykan.com>> wrote:
>
> On 28/09/15 19:14, anatoly techtonik wrote:
>
> If I understand correctly, this is the backend that uses full text
> search mechanism of PostgreSQL. Before removing that, it will be
> nice to
> have some tests that prove that search works and some test for
> additional capabilities of full text search that are being removed.
>
>
> I agree that more testing for the PostgreSQL backend would be good,
> but considering that the tsearch2 backend is essentially dead code
> that never seems to have made it to production quality, I don't
> think add tests should be a blocker to removing the backend.
>
>
> I don't know if that code works or not - that's why I asked for tests.
> Having search that finds at least some relevant info is better than no
> search at all.
All of the tests currently in test/test_tsearch2.py have equivalent
tests in test/test_postgresql.py
The tests for all the DB backends are essentially the same (the tests
case classes all inherit tests from common test classes). If there is
any new testing that is required, then those new tests are not just
specific to the PostgreSQL backend, but would be needed for all DB backends.
Removing the tsearch2 backend is not going to leave the PostgreSQL
backend with any less test coverage than the other DB backends. If there
is a need for more tests, adding them should not be a blocker for
removing the tsearch2 backend.
No one has raised a case for keeping the tsearch2 backend. Given that
the tsearch2 extension has been abandoned and that it is unclear whether
the backend was ever functional, I think it is safe to assume that we
can get rid of it and remove some useless code from the project.
SeeYa,
John. |
|
Date |
User |
Action |
Args |
2015-10-10 02:58:15 | jerrykan | set | recipients:
+ jerrykan |
2015-10-10 02:58:15 | jerrykan | link | issue2550895 messages |
2015-10-10 02:58:14 | jerrykan | create | |
|