Let the basic data structures Name, Record (and its subclasses), RRset and Zone implement Serializable to be able to serialize such objects e.g. for exchanging them by RMI.
Are these just random numbers, or were they generated somehow? This adds support for all the current subclasses of Record, but there's no documented procedure for adding support to new subclasses.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Ok, basic was the wrong word here: all data structures which have to be exchanged through different Java VMs have to be serializable. In my scenario you can edit a Zone within a client and when you hit save it will be transfered to a server which supplies it to a nameserver. So zone has to be also serializable.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Unified diff against dnsjava-2.0.6
Are these just random numbers, or were they generated somehow? This adds support for all the current subclasses of Record, but there's no documented procedure for adding support to new subclasses.
The serialVersionUID may be random. Those ones in the patch were generated by Eclipse IDE. They are required by the Serializable interface. Have a look at http://java.sun.com/javase/6/docs/api/java/io/Serializable.html
New subclasses of Record will inherit the Serializable interface from Record and just have to define a serialVersionUID.
OK. I'll apply this patch (later today), and update any classes added since the patch was generated.
Is there a reason why Zone is included? Name and Record are pretty basic data structures, but Zone isn't.
Committed.
Ok, basic was the wrong word here: all data structures which have to be exchanged through different Java VMs have to be serializable. In my scenario you can edit a Zone within a client and when you hit save it will be transfered to a server which supplies it to a nameserver. So zone has to be also serializable.
Seems reasonable. It's all committed now, and there should be a release as soon as some other patches I'm expecting come in.
Great!
Ticket moved from /p/dnsjava/patches/14/