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

Should be 'db purge' be removed? #335

Open
tonyespy opened this issue Nov 3, 2020 · 1 comment
Open

Should be 'db purge' be removed? #335

tonyespy opened this issue Nov 3, 2020 · 1 comment
Labels
bug Something isn't working

Comments

@tonyespy
Copy link
Member

tonyespy commented Nov 3, 2020

I'd like to see some discussion as whether or not the inclusion of the 'db purge' command is appropriate for edgex-cli. Most, if not all of the other commands exercise EdgeX's REST APIs.

I see that the command implements a "do you really want to do this check", which is great, however I would feel more comfortable with pointing someone to the actual db tools (e.g. redis-cli).

Maybe this could be discussed at an upcoming Core WG meeting?

Version: v1.0.0-dev.28

@tonyespy tonyespy added the bug Something isn't working label Nov 3, 2020
@difince
Copy link
Contributor

difince commented Nov 5, 2020

The initial idea was to keep the client unaware of the underlying implementation - unaware of what is the used database and still test most of the Delete exported apis . But better is to have a discussion about it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants