feat: Derive parcels in estate for performance (#837)

* feat: Derive parcels in estate for performance

* feat: Remove parcels settup in buildEstateFromNFT

* feat: Update how size and searchEstateSize are assigned

* chore: Add temp to readme

* chore: Update hashes in readme

* chore: Update goerli hashes with new deployment

* chore: Move temp under goerli

* chore: Add temp to deploy and package json

* chore: Update temp hashes with new deploy

* chore: Update hashes

Author: fzavalia

indexer/README.md

@@ -1,10 +1,17 @@
 # Blockchain indexer
 
-- Mainnet: https://thegraph.com/hosted-service/subgraph/decentraland/marketplace (QmWF9uXD9KokRU1hA5Gnmc3VKU5mPU3cAYmD15cfa8YDTT)
-- Ropsten: https://thegraph.com/hosted-service/subgraph/decentraland/marketplace-ropsten (QmfHCGhLTZV8v2duxDkKtPZKMEdJM7X8YGRj2UvqmrAUBB)
+- Mainnet: https://thegraph.com/hosted-service/subgraph/decentraland/marketplace
+  - Prev: `QmYXc2SCC85dvd3pL6yYfxCkLuaCvJaiES3pA4inGiGwej`
+  - Curr: `QmccAwofKfT9t4XKieDqwZre1UUZxuHw5ynB35BHwHAJDT`
+- Ropsten: https://thegraph.com/hosted-service/subgraph/decentraland/marketplace-ropsten
+  - Prev: `QmfHCGhLTZV8v2duxDkKtPZKMEdJM7X8YGRj2UvqmrAUBB`
+  - Curr: `QmZTADndoP4XRoWGVoQuaz8WTATx3UDXMn5SdE3GfkErkW`
 - Goerli: https://thegraph.com/hosted-service/subgraph/decentraland/marketplace-goerli
-  - Prev: `QmVJ5XudhdALBoyYg8w4bPC62E2N6LKgDteVA2NFnETsL2`
-  - Curr: `QmRNcz6TmfrFSYZcChetAiivCktGnEpAMrdRYFg74Sm9CU`
+  - Prev: `QmRNcz6TmfrFSYZcChetAiivCktGnEpAMrdRYFg74Sm9CU`
+  - Curr: `Qmds1Ut7L2Kvr5Y4ZnCvpMaomcQvhq7BpjhVPgxP5TEfhT`
+- Temp: https://thegraph.com/hosted-service/subgraph/decentraland/marketplace-temp
+  - Prev: `QmccAwofKfT9t4XKieDqwZre1UUZxuHw5ynB35BHwHAJDT`
+  - Curr: `QmYXc2SCC85dvd3pL6yYfxCkLuaCvJaiES3pA4inGiGwej`
 
 It uses [thegraph](https://thegraph.com)
 

indexer/package.json

@@ -11,7 +11,8 @@
     "deploy-studio:mainnet": "graph deploy --node https://api.studio.thegraph.com/deploy/ --ipfs https://api.thegraph.com/ipfs/ decentraland-marketplace-ethereum-mainnet",
     "deploy:ropsten": "ts-node ./scripts/buildData.ts --network ropsten && ts-node ./scripts/deploy.ts --network ropsten",
     "deploy:mainnet": "ts-node ./scripts/buildData.ts --network mainnet && ts-node ./scripts/deploy.ts --network mainnet",
-    "deploy:goerli": "ts-node ./scripts/buildData.ts --network goerli && ts-node ./scripts/deploy.ts --network goerli"
+    "deploy:goerli": "ts-node ./scripts/buildData.ts --network goerli && ts-node ./scripts/deploy.ts --network goerli",
+    "deploy:temp": "ts-node ./scripts/buildData.ts --network mainnet && ts-node ./scripts/deploy.ts --network temp"
   },
   "devDependencies": {
     "@graphprotocol/graph-cli": "^0.21.1",

indexer/schema.graphql

@@ -91,7 +91,7 @@ type Estate @entity {
   id: ID!
   tokenId: BigInt!
   owner: Account!
-  parcels: [Parcel!]
+  parcels: [Parcel!]! @derivedFrom(field: "estate")
   size: Int
   data: Data
   rawData: String

indexer/scripts/deploy.ts

@@ -3,13 +3,15 @@ import { spawn, SpawnOptions } from 'child_process'
 enum Network {
   MAINNET = 'mainnet',
   ROPSTEN = 'ropsten',
-  GOERLI = 'goerli'
+  GOERLI = 'goerli',
+  TEMP = 'temp'
 }
 
 const graphByNetwork: Record<Network, string> = {
   [Network.MAINNET]: process.env.GRAPH_NAME || 'decentraland/marketplace',
   [Network.ROPSTEN]: process.env.GRAPH_NAME || 'decentraland/marketplace-ropsten',
-  [Network.GOERLI]:  process.env.GRAPH_NAME || 'decentraland/marketplace-goerli'
+  [Network.GOERLI]:  process.env.GRAPH_NAME || 'decentraland/marketplace-goerli',
+  [Network.TEMP]:  process.env.GRAPH_NAME || 'decentraland/marketplace-temp'
 }
 
 // TODO: Handle ctrl+C

indexer/src/handlers/estate.ts

@@ -25,7 +25,6 @@ export function handleCreateEstate(event: CreateEstate): void {
   estate.tokenId = event.params._estateId
   estate.owner = event.params._owner.toHex()
   estate.rawData = data
-  estate.parcels = []
   estate.size = 0
 
   let estateData = buildData(id, data, DataType.ESTATE)
@@ -58,16 +57,12 @@ export function handleAddLand(event: AddLand): void {
 
   let estate = Estate.load(id)
 
-  let parcels = estate.parcels
-  parcels.push(parcelId)
-
-  estate.parcels = parcels
-  estate.size = parcels.length
+  estate.size += 1
 
   estate.save()
 
   let estateNFT = NFT.load(id)
-  estateNFT.searchEstateSize = parcels.length
+  estateNFT.searchEstateSize = estate.size
   estateNFT.save()
 
   let parcel = Parcel.load(parcelId)
@@ -101,17 +96,12 @@ export function handleRemoveLand(event: RemoveLand): void {
 
   let estate = Estate.load(id)
 
-  let parcels = estate.parcels
-  let index = parcels.indexOf(parcelId)
-  parcels.splice(index, 1)
-
-  estate.parcels = parcels
-  estate.size = parcels.length
+  estate.size -= 1
 
   estate.save()
 
   let estateNFT = NFT.load(id)
-  estateNFT.searchEstateSize = parcels.length
+  estateNFT.searchEstateSize = estate.size
   estateNFT.save()
 
   let parcel = Parcel.load(parcelId)

indexer/src/modules/estate/index.ts

@@ -5,7 +5,6 @@ export function buildEstateFromNFT(nft: NFT): Estate {
 
   estate.tokenId = nft.tokenId
   estate.owner = nft.owner
-  estate.parcels = []
   estate.size = 0
 
   return estate