search

jforman / binder

Django Web Admin Gui for manging BIND DNS Zones
129 stars 41 forks source link

Improved record deletion performance #20

Open Dunedan opened 9 years ago

Dunedan commented 9 years ago

Up to now when selecting multiple resource records for deletion for each of them a single message has been sent to the DNS server. This commit changes the behavior to put all requested deletions into a single message, which improves performance.

Dunedan commented 9 years ago

I've added some comments to the lines commented. Just in case they don't show up because of the updated diff.

Dunedan commented 9 years ago

I understand your concerns regarding the output formatting in code. As I don't have a good solution for that right now, let's just keep the pull request as it is (or close it if you want).