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

Fix bug that database is deleted when database having same id is tended to be added #484

Merged
merged 1 commit into from
Aug 1, 2022

Conversation

WatanabeToshimitsu
Copy link
Contributor

@WatanabeToshimitsu WatanabeToshimitsu commented Jul 29, 2022

What?

  • Fix bug that database is deleted when database having same id is tended to be added
  • Do not delete database when occur error relating adding database

Why?

Bug fixes

Screenshot or video

2022-07-29.18.56.10.mov

@WatanabeToshimitsu
Copy link
Contributor Author

WatanabeToshimitsu commented Jul 29, 2022

@yusukefs バグの修正に加えて、データベース追加時にデータベースを削除するリクエストを飛ばすこと自体も、危険性が高いので削除しましたが、どうですかね...?
このコードを書いたときは「ネットワークエラー等が起こってデータベースが追加されたかどうか怪しい場合は消した方がいいかな」とか「正しく初期化できないデータベースは消した方がいいかな」というつもりだったと記憶してるんですが、データベースが metaStore 上に追加されていればGUIから削除できるはずなので、安全をとった方がいいのかなーと

Copy link
Contributor

@yusukefs yusukefs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
安全を取って削除はユーザーに任せるのがシンプルで良いと思います。

@WatanabeToshimitsu WatanabeToshimitsu merged commit e643cc8 into master Aug 1, 2022
@d-hayashi d-hayashi deleted the fix/delete-duplicated-database branch August 5, 2022 02:10
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.

2 participants