Build #4,246

License check with RAT and Checkstyle

Build: #4246 was successful Changes by albertshau <ashau@google.com>

Build result summary

Details

Completed
Duration
4 minutes
Labels
None
Agent
bamboo-agent10
Revision
b4587276e5c7309f905c652480303c45c63794ad b4587276e5c7309f905c652480303c45c63794ad
Successful since
#4040 ()

Code commits

Author Commit Message Commit date
albertshau <ashau@google.com> albertshau <ashau@google.com> b4587276e5c7309f905c652480303c45c63794ad b4587276e5c7309f905c652480303c45c63794ad Merge pull request #11007 from cdapio/feature/CDAP-14839-spi-exception-changes
CDAP-14839 replace usage of NotFound and AlreadyExists exceptions
albertshau <ashau@google.com> albertshau <ashau@google.com> 3f491bef49190579ce6a4f0ac144738d7d670778 3f491bef49190579ce6a4f0ac144738d7d670778 CDAP-14839 replace usage of NotFound and AlreadyExists exceptions
Replaced NotFoundException and AlreadyExistsException with classes
specific to the storage SPI. This was done to prepare to move
the SPI definitions into their own module with minimal dependencies.
NotFoundException and AlreadyExistsException are in cdap-common,
which cannot be used.

Also corrected some exception handling in the ArtifactStore if
getting a StructuredTable throws a TableNotFoundException.
It was not correct to throw a NotFoundException, as that type of
error should not result in a 404, but a 500 response code.

Also removed some guava usage, as the SPI should not depend on
guava either.

JIRA issues

IssueDescriptionStatus
Unknown Issue TypeCDAP-14839Could not obtain issue details from JIRA