Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wrong sorting when using lucene fields #3042

Open
eduarddrenth opened this issue Oct 8, 2019 · 11 comments
Open

wrong sorting when using lucene fields #3042

eduarddrenth opened this issue Oct 8, 2019 · 11 comments
Assignees
Labels
investigate issues being looked at Lucene issue is related to Lucene or its integration

Comments

@eduarddrenth
Copy link
Contributor

In exist-db 5.0.0, when using an xpath in the form:

...//tei:cit[ft:query(tei:quote,(),map{"fields": ("xmlid")})]/tei:quote

and sorting in the form:

           let $hits := for $o in teidict:queryText($searchin,$searchterm)
                       order by ft:field($o,"xmlid")
                       return $o

The sorting is wrong.

A complete reproducer can be found here:

https://bitbucket.org/fryske-akademy/online-dictionaries/src/field_wrong_sorting/readme.md

@triage-new-issues triage-new-issues bot added the triage issue needs to be investigated label Oct 8, 2019
@duncdrum duncdrum added the investigate issues being looked at label Oct 28, 2019
@triage-new-issues triage-new-issues bot removed the triage issue needs to be investigated label Oct 28, 2019
@eduarddrenth
Copy link
Contributor Author

included link to source code in readme of reproducer

@dizzzz
Copy link
Member

dizzzz commented Nov 5, 2019

Hi @eduarddrenth please could you provide

  1. (parts of) the actual result
  2. (parts of) the expected result ?

@eduarddrenth
Copy link
Contributor Author

I already prepared and tested a complete reproducer with readme, isn't that enough?

@dizzzz
Copy link
Member

dizzzz commented Nov 11, 2019

Hi @eduarddrenth, we ask ppl to fill in the template (as well) because it helps to make a 1st order assessment: Can I pick this up? The template really helps. For me, personally, As I need to be picky time wise [we are all volunteers!] on the work I can pick up, for this 1st order estimation I will and can not do these kind of investigations. In the end it is often 'a waste of time' as most of the time the user has different expectations on results than I have. It results in frustrations and time late on two sides. SO filling up the template helps, you and us.

@eduarddrenth
Copy link
Contributor Author

I'll see, I already spend considerable amount of time in the reproducers, this feels a bit like administrative overhead. But it shouldn't be too hard for me to add expected and actual, so I'll find some time. A product like exist-db should be funded somehow I think, it is worth it.

@duncdrum
Copy link
Contributor

A product like exist-db should be funded somehow I think, it is worth it.

working on it, more news soon(™)

@duncdrum
Copy link
Contributor

@eduarddrenth
Copy link
Contributor Author

ok, it took me a while to reproduce, but I succeeded.

Following https://bitbucket.org/fryske-akademy/online-dictionaries/src/field_wrong_sorting/readme.md

Actual results are (starting from e on page 1):

image

Expected results (changed [ft:query(tei:quote,(),map{"fields": ("xmlid")})]/tei:quote into /tei:quote[ft:query(.,(),map{"fields": ("xmlid")})]):

image

@eduarddrenth
Copy link
Contributor Author

I seem to remember that with a smaller testset the problem does not occur

@eduarddrenth
Copy link
Contributor Author

https://opencollective.com/existdb

Nice, pitty paypal is only supported for non recurring

@eduarddrenth
Copy link
Contributor Author

just tested with 5.1.1, still reproducable

@line-o line-o added the Lucene issue is related to Lucene or its integration label Apr 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigate issues being looked at Lucene issue is related to Lucene or its integration
Projects
None yet
Development

No branches or pull requests

5 participants