Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

Remove database, provide Persistor at Start #93

Merged
merged 5 commits into from Apr 9, 2021

Conversation

djaglowski
Copy link
Member

This PR prepares the library for the use of the storage extension framework.

The idea is that state will be persisted at a higher level, and only a simple persistence interface is necessary within this library.

  • Removes the concept of a database, and the associated dependency on bbolt
  • Reworks the Persister interface, removing Sync and Load, but adding context.Context to all methods
  • Passes a Persister to each Start method (Agent.Start, Pipeline.Start, Operator.Start)
  • Moves Persister into the operator package, because it is similar to BuildContext

@djaglowski djaglowski marked this pull request as ready for review April 8, 2021 16:18
@djaglowski djaglowski merged commit a881ed8 into open-telemetry:main Apr 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants