Uploaded image for project: 'Couchbase .NET client library'
  1. Couchbase .NET client library
  2. NCBC-915 Support .Net Core / DNX Core
  3. NCBC-1115

Change NodeUnavailableException Base Class And Other Exception Issues

    XMLWordPrintable

Details

    • Technical task
    • Resolution: Fixed
    • Major
    • 2.4.0-dp1
    • None
    • library
    • SDK4: XATTR, S.App Test, Analy, SDK6: DNS-SRVd, Wrapup

    Description

      .Net Core does not include remoting or the System.Runtime.Remoting namespace. Currently, NodeUnavailableException is inherited from System.Runtime.Remoting.ServerException. This base class should be changed to a more consistent base class, such as Exception.

      ApplicationException is no longer available, either. Apparently the .Net approach has been to move away from it, and recommend inheriting your exceptions directly from Exception. There is one location in code throwing ApplicationException directly, which should be changed to Exception or a new class created. This is edge case code, so it should be safe to change without breaking compatibility.

      Attachments

        For Gerrit Dashboard: NCBC-1115
        # Subject Branch Project Status CR V

        Activity

          People

            btburnett3 Brant Burnett
            btburnett3 Brant Burnett
            Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty