Skip to content

revision管理用のテーブルを追加#60

Open
shogo82148 wants to merge 3 commits intomasterfrom
feature/revision-table
Open

revision管理用のテーブルを追加#60
shogo82148 wants to merge 3 commits intomasterfrom
feature/revision-table

Conversation

@shogo82148
Copy link
Copy Markdown
Contributor

GoogleDriveにアップロードする直前にFileIdが空のBundleを追加してrevisionをあげていましたが、
以下の問題があります

  • Upload中にFileIdが空っぽのBundleが他のユーザに見えてしまう
  • GoogleDriveへのアップロードが失敗すると、FileIdが空のまま残ってしまう
  • Bundleを削除してしまうと同じrevisionが再利用されてしまう

revisionを別テーブルに分離しbundleとは別で管理することにより、これらの問題を解消します。

Comment thread app/models/app.go
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

このSQLであれば行があるときでもない時でも使えるので上のSQLを走らせずにいきなりこちらを使えばいいと思ったのですがどうでしょうか

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

このSQLは過去との互換性のためだけに残してあります。

この管理方法では

  • revisionをインクリメントする
  • bundleをDBに挿入する

という処理を分離できません。

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

なるほど 🙆‍♀️

@mackee
Copy link
Copy Markdown

mackee commented Mar 9, 2015

見ました〜

@m0t0k1ch1
Copy link
Copy Markdown
Contributor

Revision もテーブルにひもついてはいるのでファイル分けといた方がわかりやすいかなとは思いました。IncrementRevision の中に切り出せる処理もありそう。

@m0t0k1ch1
Copy link
Copy Markdown
Contributor

どこかで見切りつけて既存の bundlerevision を drop したいすな。。

@m0t0k1ch1
Copy link
Copy Markdown
Contributor

見ました

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants