David Lutterkort

> @dotansimha that is a very good point - I hadn't thought about the fact that `Block_height` already has `number_gte`, which fits perfectly with what we want to do. I...

`table.columns` does include fulltext columns (see [here](https://github.com/graphprotocol/graph-node/blob/master/store/postgres/src/relational.rs#L1214-L1220)) but we never insert the `vid`, that is determined by the database. Are we sure that that issue is caused by the `InsertQuery`?...

> It seems that [`impl QueryFragment for InsertQuery`](https://github.com/graphprotocol/graph-node/blob/28580bb76178bde5e4174dd5f73b1aed0e706b87/store/postgres/src/relational_queries.rs#L1545) is counting the bindings correctly: one for each field plus one for the block range. > > But `InsertQuery` delegates `walk_ast` to...

My one concern with the proposed schema is the `totalCount` field on `PurposeConnection` since that can be _very_ expensive to calculate. I think it would be better to leave that...

> My reasoning for removing this restriction is that it enables easier node provisioning by enabling the use of generated Kubernetes host names to name index and query nodes. This...

[This](https://github.com/graphprotocol/graph-node/blob/master/store/postgres/src/copy.rs#L523) is wrong, it needs to be calculated from the information in `state.tables`.

Closing this since it's not been active for a while. Feel free to reopen when you're ready to work on this again

Fixed by https://github.com/graphprotocol/graph-node/pull/5075

> 2. Let operators set default history per network in `graph-node.toml` > > > ``` > [chains.mainnet] > history = 10000 > ``` I am generally ok with what's proposed...

> This is something @lutter can confirm, but afaik all the infrastructure for ongoing pruning is there and we just need to use it just like that flag does. Yes,...