News for hackers. Stuff that matters.
Re:The GitHub Availability Report by shenli3514 on Thursday July 16, 2020 @03:02AM (#23856544)
I'm the VP of Engineering from PingCAP.

As a devoted customer and a big fan of GitHub, it’s devastating to see service interruptions and our business impacted by these problems; as the team behind TiDB, we (PingCAP) believe there is something we can do to help solve the database high availability and scalability problem. We would like to propose the database team in GitHub to consider the TiDB platform. TiDB and TiKV can work as a scale-out MySQL and it has been battle-tested in all kinds of hyper-scale scenarios.

Additionally, just in case you didn’t do this, we also recommend GitHub considering the Chaos-Mesh project (https://github.com/pingcap/chaos-mesh) to do Chaos Engineering. Chaos Mesh was our internal chaos engineering platform and we open-sourced it on Dec. 31, 2019. It can be used for simulating different kinds of failures including network partition, flaky disk, node outage. You can easily use it to simulate failures in a test environment and confirm that the high availability works as expected.

Re:The GitHub Availability Report by capableweb on Thursday July 16, 2020 @04:07AM (#23856865)
If you read the published report, you'll see that only one of the issues are actually related to scalability in any way (and most newly created projects learned their lesson of not using auto-incrementing IDs for many reasons). The same goes for many companies, the downtime is not because of any scaling issues but rather maintenance, code deploys or other things where humans are involved.

So while the reward of you promoting your product on HN can be high, unless it's very specific and actually solves their need, it just looks spammy.

Re:The GitHub Availability Report by jrockway on Thursday July 16, 2020 @05:09AM (#23857147)
The big problem that I took away from the availability report was the one where they lost acknowledged writes. They might not need the scaling, but they do need the ability to lose the leader without losing writes. (It would be preferable to reject the writes and become unavailable rather than to acknowledge writes only to discard them minutes later. No external API user can handle the case where "Github acknowledged my writes, and I read them back, but now they're gone". But they can handle "Github returned 'service unavailable' when I made a write".)
Re:The GitHub Availability Report by shenli3514 on Thursday July 16, 2020 @05:31AM (#23857257)
Thanks for your comment. Yes, there is only one of the issues are actually related to scalability. So I mentioned Chaos-Mash as well. Hope it would be helpful to test large scale distributed system. Both of them can not solve all the issues. But I hope they would be helpful for some of the issues.
Re:The GitHub Availability Report by ithkuil on Thursday July 16, 2020 @03:50AM (#23856775)
Can it simulate the overflow of a 32-bit autoincrementjng primary key? Should it?
Re:The GitHub Availability Report by shenli3514 on Thursday July 16, 2020 @04:08AM (#23856869)
Do you mean Chaos-Mesh or TiDB? For Chaos-Mesh, it is not a suitable tool for deterministic unit test case (like inserting an overflow value and expect to get an error). For TiDB, it is easy to use `Alter Table` statement to enlarge the length of a integer column with zero impact on the online business.
Re:The GitHub Availability Report by capableweb on Thursday July 16, 2020 @04:13AM (#23856894)
Not sure if you're purposefully missing the question here.

ithkuil is not asking if you can change the length of the integer column but rather if you somehow would have prevented the issue GitHub saw here with hitting the limit. So something like automatically updating the length, or giving the developers warnings, or alerts of some sort.

My guess would be no, as I haven't seen this in the wild.

And if the answer is indeed "no", your comment seems again to plug something that wouldn't actually solve the problem GitHub saw here.

Re:The GitHub Availability Report by ithkuil on Thursday July 16, 2020 @04:21AM (#23856924)
What capableweb said.

Furthermore, I wonder how long does it take to enlarge a pk column using alter table on a database that has a significant fraction of 4 billion rows?

EDIT: I'm not arguing that github would not benefit from TiDB; they might be genuinely better of with it on many fronts. I just wonder if it's fair to plug a product this way

Re:The GitHub Availability Report by shenli3514 on Thursday July 16, 2020 @05:42AM (#23857310)
Thanks for the clarification. There is not automatically way to enlarge the length. And I think that is not the things should be done by the database itself. That's my own opinion. But I think it would be helpful to show a warning to DBA when the allocated value is close to the upper limit.

For the following question, the answer is in seconds. To enlarge the column length, TiDB just need to modify the meta data. "how long does it take to enlarge a pk column using alter table on a database that has a significant fraction of 4 billion rows?"

Re:The GitHub Availability Report by ithkuil on Thursday July 16, 2020 @07:06AM (#23857701)
> There is not automatically way to enlarge the length. And I think that is not the things should be done by the database itself.

Sorry; it's hard to balance terseness with clarity. What I meant is to ask whether a tool such as Chaos-Mesh could have helped the github operations team to be aware of such a failure mode by causing this kinds of problems early.

I think that's the purpose of chaos-engineering right? Some things happen too infrequently for us to plan for them, until when they finally do, causing problems that are more costly to solve than the cost of the prevention.

It's still unclear to me whether this particular failure mode is common enough to be part of what a chaos-engineering tool explores by default; after all, not all databases around contain billions of rows. If engineers have to explicitly enable such a test because they are afraid of what would happen if a PK overflows, well, then probably they would take other preventive measures so that the problem doesn't happen in the first place (either preventively upgrading the size of the counter, or at least put some monitoring). I think what happened here is that a solution was designed when the database was way smaller; the user base grew and grew and this particular failure mode was overlooked (because there are many many other things to care about!).

(I write this because I'm genuinely curious about what other people think about this topic, not to bash on parent for a shameless plug; I mentioned that only as a side comment; I heard good things about TiDB, I wish you well)

Re:The GitHub Availability Report by user5994461 on Thursday July 16, 2020 @07:16AM (#23857759)
>>> It's still unclear to me whether this particular failure mode is common enough

It is very common. Had the same thing happen in my company last year, on multiple systems! The organization was reaching a decade old, nobody thought of that when the table were initially created.

Re:The GitHub Availability Report by ithkuil on Thursday July 16, 2020 @08:32AM (#23858307)
Yeah I now realize it's not necessarily 4billion rows actively used, but just 4billion unique keys being allocated
Re:The GitHub Availability Report by capableweb on Thursday July 16, 2020 @06:46AM (#23857610)
> we believe there is something we can do to help solve the database high availability and scalability problem

> There is not automatically way to enlarge the length

> I think that is not the things should be done by the database itself

> would be helpful to show a warning to DBA

So, in the end, TiDB does nothing to actually solve the problem they experienced but you still feel comfortable offering them your own solution? Sounds like you're a VP of Marketing, not a VP of Engineering. Leaves a very sour taste when spamming a forum like this. I hope your bosses don't read HN and sees your comments plugging TiDB everywhere where it's not relevant.

Re:The GitHub Availability Report by neilhan on Sunday July 19, 2020 @12:13AM (#23886941)
well said, we will take note; We were just trying to help anything related to MySQL's scalability
Re:The GitHub Availability Report by ithkuil on Thursday July 16, 2020 @06:54AM (#23857643)
> TiDB just need to modify the meta data.

thanks, that's interesting. So existing data using the previous 32-bit encoded values happily coexist with the new data being written after the migration?