-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
testing generic table #3
Comments
For that I would also suggest some kind of cli utility to insert into the database |
yep. Or, at the least, a converter. Within the context of CoreDNS we should aim to keep it read-only, so another process fills the database (from an AXFR/IXFR?) and multiple coredns instances serve from it |
Yep, definitly external to Coredns, but using your dns library with all the objects in there. A mapper/encode could take out the common columns, and build a wrapper object around with created_at / updated_at / zone/ id information |
yes, at least that what I'm thinking here. You basically have a dns.MsgHeader on file in the rdata by decoding the Gob for this specific RR. |
for that I would think about a config option: schema explicit = every record has a separate table so you can decide what you prefer |
Sure, it's your code :) I just think a generic database is easier from an ops standpoint. |
I might be worth testing with a database that stores the rdata in a generic way and sees how much that impact the performance.
What this generic rdata should look like is an interesting question, esp. coming from CoreDNS that uses go-dns for internal representation. May use https://golang.org/pkg/encoding/gob/ ?
This request comes mostly from my spider sense going off when you need to "upgrade your database" to implement a new record.
The text was updated successfully, but these errors were encountered: