Tao Zhu

Results 62 comments of Tao Zhu

proposed a fix at #35012. Tested with testnet snapshot to confirm hash matches; Going to do test with mainnet ledger too

confirmed bankhash matches with testnet ledger. Going to confirm it does the same with mainnet ledger.

Close it for now, issue #https://github.com/solana-labs/solana/issues/34982 tracks it. Removing round() needs a feature gate, which requires an additional parameter to `calculate_fee()` which is an exposed public api, making it a...

Close this without merge; we decided to keep original `calculate_fee(...)` public api unchanged in #34757, so instead of one big-bang removal, I will break them into smaller pieces as it...

runtime-transaction is few steps from action. I'm leaning to wait for it if you can.

Adding tx signature fee into priority calculation makes sense from validator perspective, but want to point out the cost impact to transaction submitters. The priority order changed from `cu_price` to...

> > Another thought is: due to the impact to end user, might be good idea to have community sign up before merge, wdyt? > > what do you mean...

as Draft until rebaseed after #34821 is merged

> Also, if im reading it correctly, the banking stage thread first filters out tx based on max block cu limits and then tries to acquire the write lock for...