Task #1642
Story #1639: use CNCore/CNode for all ObjectFormat methods
use ObjectFormatCache when implementing CNode methods
0%
Description
Since ObjectFormatCache already implements the CNCore methods, we can just delegate to that class from CNode (the libclient implementation of CNCore).
Optionally merge ObjectFormatCache with ObjectFormatServiceImpl if they can logically/feasibly be combined into a single class.
History
#1 Updated by Chris Jones over 13 years ago
- Status changed from New to Rejected
CNode already implements listFormats() and getFormat() by calling the D1RestClient, and the ObjectFormatCache uses this implementation to cache the list. So, I think the functionality is already there, but is the reverse of what is suggested here. Also, ObjectFormatCache and ObjectFormatServiceImpl can't be merged because there would be a d1_common_java dependency on d1_libclient_java in the tests. We've discussed this and decided to keep them separate. I'll reject this for now unless CNode should depend on ObjectFormatCache instead of the other way around.
