This is another post (<Rant>) about WCF default behavior and how it can make the life of developers miserable ( you can also check out “WCF defaults limit scalability” and “Another WCF gotcha - calling another service/resource within a call”)
Anyway, the trigger for this is a post by Ayende called “WCF works in mysterious ways”. Ayende posted some code he wrote which was throwing a serialization exception. You can see his post for the full code, but in a nut shell he was defining a large object graph (8192 objects that contain other objects) and was trying to send that over the wire. Here’s a short excerpt from the service definition:
1: [ServiceBehavior(
2: InstanceContextMode = InstanceContextMode.Single,
3: ConcurrencyMode = ConcurrencyMode.Single,
4: MaxItemsInObjectGraph = Int32.MaxValue
5: )]
6: public class DistributedHashTableMaster : IDistributedHashTableMaster
7: {
8: private readonly Segment[] segments;
9:
10: public DistributedHashTableMaster(NodeEndpoint endpoint)
11: {
12: segments = Enumerable.Range(0, 8192).Select(i =>
13: new Segment
14: {
15: AssignedEndpoint = endpoint,
16: Index = i
17: }).ToArray();
18: }
19:
20: public Segment[] Join()
21: {
22: return segments;
23: }
24: }
25:
26: [ServiceContract]
27: public interface IDistributedHashTableMaster
28: {
29: [OperationContract]
30: Segment[] Join();
31: }
32:
33: public class NodeEndpoint
34: {
35: public string Sync { get; set; }
36: public string Async { get; set; }
37: }
38:
39: public class Segment
40: {
41: public Guid Version { get; set; }
42:
43: public int Index { get; set; }
44: public NodeEndpoint AssignedEndpoint { get; set; }
45: public NodeEndpoint InProcessOfMovingToEndpoint { get; set; }
46:
47: public int WcfHatesMeAndMakeMeSad { get; set; }
48: }
As you can see in line 4 – the service is properly decorated with an attribute to enlarge the number of objects in graph. so looking at the code I initially suggested he add a few ServiceKnowType and DataContract/DataMember attributes on the data classes (as the serialization sometimes needs some guidance. After that didn’t help I actually ran the code and then I noticed that the code was missing setting that same attribute – on the client side. So to fix the problem, the client side code below
1: var channel =
2: new ChannelFactory<IDistributedHashTableMaster>(binding, new EndpointAddress(uri))
3: .CreateChannel();
4: channel.Join();
1: var channelFactory =
2: new ChannelFactory(binding, new EndpointAddress(uri));
3:
4: foreach (var operationDescription in channelFactory.Endpoint.Contract.Operations)
5: {
6:
7: var dataContractBehavior =
8:
9: operationDescription.Behaviors[typeof(DataContractSerializerOperationBehavior)]
10:
11: as DataContractSerializerOperationBehavior;
12:
13: if (dataContractBehavior != null)
15:
16: dataContractBehavior.MaxItemsInObjectGraph = int.MaxValue;
17:
20: }
21: var channel=channelFactory.CreateChannel();
22: channel.Join();
The main problem I find with this piece of code is the fact that it is needed at all. As the post’s title suggest I find this behavior greatly affects the loose coupling of anything that uses WCF (services or other components).
WCF requires that any change you make to the channel on the server side would be reflected in the channel on each and every client (e.g. we have a similar setting where we enlarge message sizes for webHttpBinding and there are many other such examples).
Sure, you say, that is just like adding a new field in the contract isn’t it? – Well no it isn’t since unlike anything else which appears in the (verbose as it is) SOAP contract these changes in default values, which are purely a WCF design choice, are not documented. Again, the changes in default values are not part of the contract. These are things you need to remember to pass on to you service consumer. So not only do I pay the overhead of having an explicit contract (e.g. vs. REST) – it really doesn’t work. It means that two components who use the same contract may not be interchangeable if one returns more data (in this case). It means that the two sides are coupled by the need to change these defaults and for what? WCF is smart enough to know how long is the message; WCF is smart enough to handle the message (if I encourage it by setting a behavior) why can’t it add 2 and 2 by itself?
Sometimes I just wish WCF had a TrainingWheels or DemosOnly attribute I could just set to false and make all this crap go away…
</Rant>
Subscribe to RSS headline updates from: