issue_comments: 500433806
This data as json
html_url | issue_url | id | node_id | user | created_at | updated_at | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
https://github.com/ACEmulator/ACE/pull/1985#issuecomment-500433806 | https://api.github.com/repos/ACEmulator/ACE/issues/1985 | 500433806 | MDEyOklzc3VlQ29tbWVudDUwMDQzMzgwNg== | 51647865 | 2019-06-10T14:19:19Z | 2019-06-10T14:19:19Z | NONE | The use case is that it makes both storing and reading the IP address easier. The performance gain from VARBINARY is negligible and makes the IP address unreadable when you want to simply look at the account table and see which IP belongs to which account. I think it's silly to block this PR because it's not modern enough for you. Keeping things simple is the best approach, especially for a community driven open source emulator project. | {"total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0} | 454022098 |