Uploaded image for project: 'C# Driver'
  1. C# Driver
  2. CSHARP-5440

Error in DictionarySerializer

    • Type: Icon: Bug Bug
    • Resolution: Works as Designed
    • Priority: Icon: Major - P3 Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Serialization
    • None
    • None
    • Dotnet Drivers
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?
    • None
    • None
    • None
    • None
    • None
    • None

      Summary

      After upgrading to MongoDb driver version 3 our software starts throwing NotImplementedException

      The problem is in class  DictionarySerializerBase<TDictionary, TKey, TValue>

       

      If you check the above implementation we have a couple of bugs. The first one is that the throw new notimplementedexception must use the very same message of the Obsolete attribute, or we, as user, receive a generic NotImplementedException without any clue.

      But the bug is that, the attribute told me to call CreateAccumulator, but CreateAccumulator call the very obsolete method with a pragma disable, so it will throw error. The net result is that I have custom DictionarySerializer that are not working anymore because that class is now broken.

       

      Please provide the version of the driver. If applicable, please provide the MongoDB server version and topology (standalone, replica set, or sharded cluster).

      Version 3

            Assignee:
            boris.dogadov@mongodb.com Boris Dogadov
            Reporter:
            alkampfer@nablasoft.com Gian Maria Ricci
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:
              None
              None
              None
              None