search

jambonz / realtimedb-helpers

utility functions for querying jambonz redis database
0 stars 9 forks source link

feature: add new command to flush tts cache #21 #23

Closed tesarm closed 1 year ago

tesarm commented 1 year ago

I'm not sure about number of possible tts keys in cache and performance/requirements. Do we need to rewrite purge of all to smaller chunks ?

davehorton commented 1 year ago

I'm not sure about number of possible tts keys in cache and performance/requirements. Do we need to rewrite purge of all to smaller chunks ?

Hmm, could you run a test -- maybe just add to the test cases -- to see how long it takes to delete 100, 1k, and 10K keys? I think 95% of the systems would have 1k or less

tesarm commented 1 year ago

deleting large volumes is fine. It's the saving that slows it down. do you want me to to keep this test and push it? It slows down test execution as it is saving one record per time.

purged large number of records - at least 100000 records, which took 68371.10ms to save and 387.36ms to purge

purged large number of records - at least 10000 records, which took 7131.50ms to save and 56.18ms to purge

purged large number of records - at least 1000 records, which took 760.61ms to save and 4.86ms to purge

purged large number of records - at least 100 records, which took 82.15ms to save and 2.02ms to purge

davehorton commented 1 year ago

OK I think we are good then as is.