[Bug]: Removal of metadata key about chroma

[tazarov]

@GauravWaghmare, take a look at the CIP PR for this - #1636. There is a bit more "nuance" to things.

I'd love your input on the CIP.

[GauravWaghmare]

@tazarov If I understand correctly this is happening because we're only inserting metadata when updating records and not touching the existing metadata. Does deleting the older metadata for the record and then inserting the new metadata sound like a good approach? If yes, can raise a PR.

[GauravWaghmare]

@tazarov Why should the behaviour for metadata update be any different from document update?

[tazarov]

@GauravWaghmare, to accommodate different use cases of how people update metadata. Users will update a document in a single overwriting operation. The intent is clear, and they don't necessarily need to know what is in the document to make the update. Metadata differs significantly, depending on the use case. The PR addresses as many use cases as I could think of:

• Clear the metadata
• Partial update
• Full overwrite of metadata

Please elaborate on your thoughts in the PR if you feel the above use cases can somehow be merged into one or if you have a different idea about the implementation.