search

msolters / Tiles

Dynamic CV and Resume app
http://tilesjs.meteor.com
0 stars 1 forks source link

Don't backup 'searchable content' #103

Closed epheterson closed 9 years ago

epheterson commented 9 years ago

This should be generated and stored on the internal db only, there's no benefit that I can tell to have the user backup and restore this information.

msolters commented 9 years ago

No, this should definitely be backed up. Searchable content is not a backend process, believe it or not. It's generated by the browser. I could use regex to strip out the HTML but it's very, very expensive for a server to do REGEX (in terms of CPU) and it always has a failure rate.

The only thing that is near-perfect at getting the human readable text from a chunk of HTML is, I'm sure you'll appreciate this logic -- a web browser. It's like...what it's made for.

So in other words getting searchable content is a totally client-side thing, not server-side.

epheterson commented 9 years ago

Couldn't it re-render it on import? This will essentially double the side of backups!

msolters commented 9 years ago

No, it will double database write-time. Write to DB is the single most expensive thing to do with a server. The increase in size is trivial compared to that throughput cost.

epheterson commented 9 years ago

Okay, you're the magician