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

参考実装移植 #234

Closed
5 tasks done
catatsuy opened this issue Aug 17, 2019 · 10 comments
Closed
5 tasks done

参考実装移植 #234

catatsuy opened this issue Aug 17, 2019 · 10 comments
Labels
移植作業 他言語への移植作業

Comments

@catatsuy
Copy link
Member

catatsuy commented Aug 17, 2019 •
edited by walf443
Loading

注意点

  • initializeの仕様はまだ変わるかもしれません
  • 他のエンドポイントは基本変更はない予定です（ならいいな）
  • フロントエンドはまだ全部実装されていません
  • READMEを必ず読んでください

移植状況

Goがマスター

  • Perl
  • PHP
  • Ruby
  • Python
  • Node.js
@catatsuy
Copy link
Member Author

早速こういう変更を入れました。今後は変更があればそのPRをここに貼っていきます。 https://github.com/catatsuy/isucon9-qualify/pull/251

@sota1235 sota1235 added the 移植作業 他言語への移植作業 label Aug 21, 2019
@catatsuy
Copy link
Member Author

ベンチマーカーで検証できないことが判明したので、こういう変更を入れました。 https://github.com/catatsuy/isucon9-qualify/pull/263

@catatsuy
Copy link
Member Author

UserAgentがデフォルトなのはさすがに厳しかったので設定しました。 https://github.com/catatsuy/isucon9-qualify/pull/282

@catatsuy
Copy link
Member Author

これをmergeしましたが、まだ決定ではないです。 https://github.com/catatsuy/isucon9-qualify/pull/310

This was referenced Aug 25, 2019
Merged
Closed
@catatsuy
Copy link
Member Author

https://github.com/catatsuy/isucon9-qualify/blob/master/webapp/go/main.go#L2283-L2294

/reportsは一旦これで確定にします。よろしくお願いします。

@kazeburo
Copy link
Collaborator

kazeburo commented Sep 2, 2019

SQLのミスがあり、後ほどまとめて修正します
catatsuy@6a4004d

@catatsuy
Copy link
Member Author

catatsuy commented Sep 2, 2019

https://github.com/catatsuy/isucon9-qualify/pull/423 上の件です。このPRをマージしました。修正はこちらでやる予定です。

@catatsuy
Copy link
Member Author

catatsuy commented Sep 2, 2019

https://github.com/catatsuy/isucon9-qualify/pull/429 こちらのバグ修正をしました。こちら問題ないか確認してもらえると助かります。

@catatsuy
Copy link
Member Author

catatsuy commented Sep 3, 2019

https://github.com/catatsuy/isucon9-qualify/pull/463 仕様変更を入れました。修正はこちらでやる予定です。

@catatsuy
Copy link
Member Author

catatsuy commented Sep 4, 2019

全言語完了！

@catatsuy catatsuy closed this as completed Sep 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
移植作業 他言語への移植作業
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kazeburo @catatsuy @sota1235