Skip to content
Snippets Groups Projects
Commit 267aacf3 authored by Jan Kadlec's avatar Jan Kadlec
Browse files

Merge branch 'bulk_updates' into 'master'

Bulk updates

Pending:
 - ~~renaming (update vs. request vs. query ...)~~
 - ~~proper logging (waiting for !250)~~
 - ~~`rrset_is_nsec3_rel()` and `mm_realloc` copypasta~~
 - partial zone adjustment - next MR

Pros:
 - faster for lots of small updates
 - improved terrible empty node cleanup (now takes time proportional to size of update, not size of the zone)
 - spinlocks instead of mutexes for ddns q access (faster for this case + less memory :fisted_hand_sign: )

Cons:
 - if one transfer fails (no matter the reason, except for acl issues), they all fail
 - might actually be slower for small singular updates, but oh well

Not sure if good or bad:
 - most of logging is lost (only logs errors in detail, i.e. who sent the faulty update)

See merge request !251
parents cafb82b5 78a2e270
No related branches found
No related tags found
No related merge requests found
Showing
with 743 additions and 363 deletions
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment