fix(eventindexer): remove nft metadata indexing, far too slowy and bu…

…g ridden right now (#18126) Co-authored-by: David <david@taiko.xyz>
taikoxyz · Sep 18, 2024 · f0ab1e1 · f0ab1e1
1 parent c2f59c3
commit f0ab1e1
Show file tree

Hide file tree

Showing 27 changed files with 24 additions and 767 deletions.
diff --git a/.github/workflows/eventindexer.yml b/.github/workflows/eventindexer.yml
@@ -44,6 +44,9 @@ jobs:
         with:
           access_token: ${{ github.token }}
 
+      - name: Install Git
+        run: sudo apt-get update && sudo apt-get install -y git
+
       - uses: actions/checkout@v4
       - uses: actions/setup-go@v5
         with:

diff --git a/packages/eventindexer/api/api.go b/packages/eventindexer/api/api.go
@@ -76,11 +76,6 @@ func InitFromConfig(ctx context.Context, api *API, cfg *Config) error {
 		return err
 	}
 
-	nftMetadataRepository, err := repo.NewNFTMetadataRepository(db)
-	if err != nil {
-		return err
-	}
-
 	ethClient, err := ethclient.Dial(cfg.RPCUrl)
 	if err != nil {
 		return err
@@ -89,7 +84,6 @@ func InitFromConfig(ctx context.Context, api *API, cfg *Config) error {
 	srv, err := http.NewServer(http.NewServerOpts{
 		EventRepo:        eventRepository,
 		NFTBalanceRepo:   nftBalanceRepository,
-		NFTMetadataRepo:  nftMetadataRepository,
 		ERC20BalanceRepo: erc20BalanceRepository,
 		ChartRepo:        chartRepository,
 		Echo:             echo.New(),

diff --git a/packages/eventindexer/errors.go b/packages/eventindexer/errors.go
@@ -12,10 +12,6 @@ var (
 		"ERR_NO_NFT_BALANCE_REPOSITORY",
 		"NFTBalanceRepository is required",
 	)
-	ErrNoNFTMetadataRepository = errors.Validation.NewWithKeyAndDetail(
-		"ERR_NO_NFT_METADATA_REPOSITORY",
-		"NFTMetadataRepository is required",
-	)
 	ErrNoStatRepository = errors.Validation.NewWithKeyAndDetail(
 		"ERR_NO_STAT_REPOSITORY",
 		"StatRepository is required",

diff --git a/packages/eventindexer/indexer/fetch_nft_metadata.go b/packages/eventindexer/indexer/fetch_nft_metadata.go
diff --git a/packages/eventindexer/indexer/index_erc20_transfers.go b/packages/eventindexer/indexer/index_erc20_transfers.go
@@ -57,7 +57,7 @@ func (i *Indexer) indexERC20Transfers(
 }
 
 // isERC20Transfer determines whether a given log is a valid ERC20 transfer event
-func (i *Indexer) isERC20Transfer(ctx context.Context, vLog types.Log) bool {
+func (i *Indexer) isERC20Transfer(_ context.Context, vLog types.Log) bool {
 	// malformed event
 	if len(vLog.Topics) == 0 {
 		return false