datawave icon indicating copy to clipboard operation
datawave copied to clipboard

Hittermtype2

Open tomnelson opened this issue 2 years ago • 1 comments

Added HitTermType and changed hit list code to use it instead of Content for hit terms. The motivation for this was as follows:

  • I saw that the document after the serialization and deserialization did not match the original for the HIT_TERM key/value. The existing code puts the accumulo key into the value of the HIT_TERM element, but that is lost during serialization/deserialization. This was evident while comparing to another effort to use json serialization. I felt that the use of Content may not have been a good choice, however changing to just use TypeAttribute with a StringType was a problem because it was difficult to differentiate between the hit term items and the actual query return items. I added a HitTermType that extends StringType. This class should probably live in datawave-type-utils in the future, when that project is ready for a release. At that time, the imports can be changed here to use that version.

tomnelson avatar Mar 10 '22 19:03 tomnelson

@ivakegg , if you're ok with @tomnelson 's updates I think we can get this merged

keith-ratcliffe avatar Aug 29 '22 22:08 keith-ratcliffe