iotexproject / iotex-core

Official implementation of IoTeX blockchain protocol in Go. An ultra-efficient EVM blockchain offering 1000 TPS with instant 1-block finality. Perfect for DeFi, DePIN tokenomics, Identities, and any trusted logic requiring Web3 composability
https://iotex.io
Apache License 2.0
1.55k stars 324 forks source link

[blockindex] remove deprecated DeleteTipBlock #4249

Closed envestcc closed 4 months ago

envestcc commented 5 months ago

Description

DeleteTipBlock has been deprecated, removing it from blockindexer

Fixes #(issue)

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Test Configuration:

Checklist:

sonarcloud[bot] commented 5 months ago

Quality Gate Failed Quality Gate failed

Failed conditions
8.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

dustinxie commented 5 months ago

should have more places to remove this func, you can refer to earlier #3854

dustinxie commented 5 months ago

we have 2 earlier PR (#2647 and #3854) trying to clean-up this.

IMO, DeleteTipBlock is a wrong method to start with, so it should be deleted. I propose let's use the earliest #2647 to finish this off. @Liuhaai let us know

envestcc commented 4 months ago

we have 2 earlier PR (#2647 and #3854) trying to clean-up this.

IMO, DeleteTipBlock is a wrong method to start with, so it should be deleted. I propose let's use the earliest #2647 to finish this off. @Liuhaai let us know

Okay, use the previous and close this one