Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

move temporary files from .dvc to .dvc/tmp #3666

Closed
efiop opened this issue Apr 22, 2020 · 1 comment · Fixed by #3725
Closed

move temporary files from .dvc to .dvc/tmp #3666

efiop opened this issue Apr 22, 2020 · 1 comment · Fixed by #3725
Assignees
@efiop
Labels
enhancement Enhances DVC
Milestone
Incompatible

Comments

@efiop
Copy link
Contributor

efiop commented Apr 22, 2020

As requested by @dmpetrov in #3366 , having state db like that in the open might be badly perceived by the users lurking around their repo. We've seen people think badly about dvc because they think we use sqlite db as a core subsystem. Let's hide these files

  • state
  • updater
  • updater.lock
  • lock
    to .dvc/tmp.
@triage-new-issues triage-new-issues bot added the triage Needs to be triaged label Apr 22, 2020
@efiop efiop mentioned this issue Apr 22, 2020
Closed
@efiop
Copy link
Contributor Author

efiop commented Apr 22, 2020

Since we will be releasing 1.0 soon, might even do that in a non-backward compatible way to simplify the logic.

@efiop efiop added this to the 1.0.0 milestone Apr 22, 2020
@efiop efiop added the enhancement Enhances DVC label Apr 22, 2020
@triage-new-issues triage-new-issues bot removed the triage Needs to be triaged label Apr 22, 2020
@weekly-digest weekly-digest bot mentioned this issue Apr 26, 2020
Closed
@efiop efiop self-assigned this May 3, 2020
efiop added a commit to efiop/dvc that referenced this issue May 3, 2020
@efiop
dvc: move temporary files to .dvc/tmp
ff6b2aa 
Fixes iterative#3666
@efiop efiop mentioned this issue May 3, 2020
Merged
3 tasks
efiop added a commit that referenced this issue May 3, 2020
@efiop
dvc: move temporary files to .dvc/tmp (#3725)
4e9b389 
Fixes #3666
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@efiop efiop

Labels
enhancement Enhances DVC
Projects
None yet
Milestone
Incompatible
Development

Successfully merging a pull request may close this issue.

dvc: move temporary files to .dvc/tmp efiop/dvc
1 participant
@efiop