Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Matching needs path support #320

Closed
anthony-gray opened this issue May 22, 2019 · 4 comments
Closed

Matching needs path support #320

anthony-gray opened this issue May 22, 2019 · 4 comments
Labels
enhancement New feature or request
Milestone

Comments

@anthony-gray
Copy link

Matching does not support matching by path, but only element name. So, an expression like identifiers/identifierRecord[identifierTypeCode='MEDICAID_ID']/identifier is not supported. This means a data model itself needs to be changed in such cases to accommodate this shortcoming (e.g. add a uniquely named medicaidId element to every document).

@ryanjdew
Copy link
Contributor

Thank you for the feedback. I will pursue adding this feature in a future release, with the understanding that this will require a path range index (http://docs.marklogic.com/guide/admin/range_index#id_40666) and will only work with exact matches (or only as flexible as collations will allow: http://docs.marklogic.com/guide/search-dev/encodings_collations#id_24886).

Are you currently using Smart Mastering as part of Data Hub 5 or as an independent library? Thanks!

@ryanjdew ryanjdew added the enhancement New feature or request label May 23, 2019
@anthony-gray
Copy link
Author

anthony-gray commented May 23, 2019

I’m using it independently. I do not intend to use DHF. In addition to path support, in general range indexes (element and path) should be supported. When dealing with 100K+ documents this makes a significant difference. However, I have filed a separate issue related to performance anyway.

Fuzzy matching is feasible for range indexes in the same way as you currently implement double-metaphone by looking up element values similar to a given value and use these values to query. I have done this already in our mastering solution, which works well and performs well.

@ryanjdew
Copy link
Contributor

This will be possible in 1.3.1 with a path range index and a cts:path-reference being associated with the property.

@anthony-gray
Copy link
Author

Nice. I will have to test this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants