Dataset framework has code duplication

Description

It seems that all code paths exists twice: once in the InMemoryDatasetOpExecutor, and once in the DatasetAdminOpHttpHandler.

For example, the logic to create a dataset exists in both places. But apparently all other methods, too.

Release Notes

None

Activity

Show:

Priyanka NambiarMarch 26, 2015 at 2:45 AM

Deprioritizing for 3.0

Terence YimMarch 17, 2015 at 1:35 AM

The Dataset framework is not following the service pattern that used everywhere in CDAP. It should have:

DatasetFramework interface, defining operations contract
DatasetFrameworkImpl implementation of the framework service
DatasetFrameworkClient Network client implementation of the framework interface
DatasetFrameworkHandler Http handler that act as a RPC translation layer to relay remote calls to DatasetFrameworkImpl.

Won't Fix
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Labels

Affects versions

Components

Priority

Created March 17, 2015 at 12:33 AM
Updated June 9, 2020 at 1:27 AM
Resolved June 9, 2020 at 1:27 AM