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.

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

[Session] セッションやキューを読み書きするリポジトリを実装 #14

Closed
1 task
p1ass opened this issue Apr 16, 2020 · 8 comments
Closed
1 task
Assignees
Labels
feature good first issue Good for newcomers

Comments

@p1ass
Copy link
Member

p1ass commented Apr 16, 2020

概要

セッションやキューを読み書きするリポジトリを実装する。

要件

すこし重たいので、テーブル設計とrepositoryの実装でPRを分けたほうが良いです。

  • IDを指定してセッションを取得できる
    • 基本情報
    • セッションの作成者
    • 現在の再生状況
  • 新規でセッションを作成できる
  • キューに曲を追加できる (削除はできなくてよい)

API仕様

API仕様をみつつ必要なデータベースのカラムを考える。

タスク

  • テーブル設計
@p1ass
Copy link
Member Author

p1ass commented Apr 30, 2020

これtrack情報を何処までDB側に保存するか悩むな

@sanposhiho
Copy link
Member

スクリーンショット 2020-05-05 4 05 58
一旦テーブル設計のレビューお願いします

@p1ass
Copy link
Member Author

p1ass commented May 5, 2020

  • SMALLINT使ってる部分あるけど、結局アプリケーション側ではint32で扱うのでINTのままでも良いかなぁって思いました!データサイズ削減以外になにか理由ってある?
  • あんまり記号の意味が分かってないんだけど、tracksののプライマリキーってuriであってる?もしそうだとすると、「ある曲は一つのセッションにしか属せない」状態になってしまって、このWebサービスを通して1回しかその曲を再生出来ない状況になりそうだなぁと思いました。

@sanposhiho
Copy link
Member

sanposhiho commented May 5, 2020

結局アプリケーション側ではint32で扱うのでINTのままでも良いかな
データサイズ削減以外になにか理由ってある?

🙆‍♂️
特になかったです

もしそうだとすると、「ある曲は一つのセッションにしか属せない」状態になってしまって

脳死でuriをPKにしてたんですけど、確かにそうですね…直しておきます

@sanposhiho
Copy link
Member

すこし重たいので、テーブル設計とrepositoryの実装でPRを分けたほうが良いです。

っていうのはSQLとrepositoryでそれぞれ別のPR出した方が🙆‍♀️ってことであってますか?

@p1ass
Copy link
Member Author

p1ass commented May 5, 2020

意図としてはリポジトリの実装をしてからテーブル設計のやり直しをすると手戻りが大きいので、途中でレビューを挟みたいという意図だったので、このままテーブル設計をissue側で議論するなら特に分けなくても大丈夫です!🙆‍♂️

@sanposhiho
Copy link
Member

結局スクショだけだったらわかりにくい部分もあるかなって思ったんでいったんPR出しましたー🙏

@p1ass
Copy link
Member Author

p1ass commented May 18, 2020

一旦終わったのでcloseします。
Thx @sanposhiho

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants