Should we have a method to merge two records or is this too fancy?

dwyl / esta

:mag: Simple + Fast ElasticSearch Node.js client. Beginner-friendly defaults & Heroku support :white_check_mark: :rocket:

49 stars 10 forks source link

Should we have a method to merge two records or is this too fancy? #88

Closed nelsonic closed 9 years ago

nelsonic commented 9 years ago

Given that we are creating a record and then only updating certain keys on that record as part of Tudo, we currently have to send the entire record just to update a single key. Would it make sense to have the equivalent the following SQL command:

UPDATE tweets
SET retweet_count = 17
WHERE id = 123

Or... is this complicating things?

nelsonic commented 9 years ago

For now lets leave record merging to the application using _esta_ until we get a specific _request_ for this feature (seconded by someone using the module in "the wild") :+1: closing.