Make UpdateDriver::parse() throw an exception instead of returning error Status

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Fixed
    • Priority: Major - P3
    • 4.1.1
    • Affects Version/s: None
    • Component/s: None
    • Fully Compatible
    • Query 2018-06-18, Query 2018-07-02, Query 2018-07-16
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Currently, UpdateDriver::parse() has a Status return type and can return an error Status, but it can also propagate exceptions from UpdateDriver::parseUpdateExpression(). Because we are moving towards exception throwing for indicating errors, we should modify UpdateDriver::parse() so that it returns void and uasserts at places where it currently returns an error.

      As far as we know, UpdateDriver::parse() call sites behave correctly in cases where exceptions are thrown, but we should probably audit them anyway when making this change.

            Assignee:
            Minji Kim (Inactive)
            Reporter:
            Justin Seyster
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: