Re: [GSOC Patch v4 0/4] Move generation, graph_pos to a slab
- From
Derrick Stolee <stolee@gmail.com>
- Date
- Jun 19, 2020, 13:59 UTC
- Message-ID
- <d5131361-2945-3daa-d91c-67761908b8ef@gmail.com>
- In-Reply-To
- <20200617091411.14650-1-abhishekkumar8222@gmail.com>
On 6/17/2020 5:14 AM, Abhishek Kumar wrote:
Show 7 quoted lines
> The struct commit is used in many contexts. However, members > `generation` and `graph_pos` are only used for commit graph related > operations and otherwise waste memory. > > This wastage would have been more pronounced as we transition to > generation nuber v2, which uses 64-bit generation number instead of > current 32-bits.
Thanks, Szeder (CC'd) for the quality review in the previous versions. I manually built and tested all of the patches here and verified they passed all tests.
I think this series is in good shape.
Thanks, -Stolee