Fix error message for an when a database connection cannot be opened or tested

Description

Steps to repro:

Test Connection

  1. Upload postgresql driver

  2. Add postgresql connection, select the installed postgresql driver

  3. In another window, open CDAP and delete the installed driver.

  4. Press test connection - you will see the error with the exception class name

Open Connection

  1. Upload postgresql driver

  2. Add postgresql connection, select the installed postgresql driver

  3. Press Save, connection will be created

  4. Delete the driver

  5. Select the connection, you will see the same error with the exception class name

 

Release Notes

None

Attachments

2
  • 05 Sep 2019, 01:24 PM
  • 29 Aug 2018, 12:37 AM

Activity

Show:

Bhooshan Mogal September 18, 2018 at 11:52 PM

Needs to be fixed at multiple places. Will take up in 5.2

Albert Shau August 29, 2018 at 12:43 AM

I assume the desired behavior is that the exception class name should not be in the error message right?

This happens whenever an exception is propagated like:

catch (IllegalArgumentException e) { throw new RuntimeException(e); }

instead of:

catch (IllegalArgumentException e) { throw new RuntimeException(e.getMessage(), e); }
Cannot Reproduce
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Components

Fix versions

Priority

Created August 29, 2018 at 12:38 AM
Updated September 5, 2019 at 1:24 PM
Resolved September 5, 2019 at 1:24 PM