search

dewi-alliance / dewi-etl

Publicly available Helium blockchain data - https://etl.dewi.org
MIT License
9 stars 1 forks source link

Convert rewards_by_day.time field to a timestamp column #22

Open jamiew opened 3 years ago

jamiew commented 3 years ago

Currently a bigint storing a unixtime

Using timestamp (or time with zone) would allow us to use Metabase's date-filtering logic out-of-the-box

cc @dansku

jamiew commented 3 years ago

BTW I wonder if we would also want to restrict visibility for the rewards table? Nobody really needs <day granularity, and people querying rewards directly will only find a dead-end. I wouln't be surprised if it started causing performance hiccups by pushing better things out of memory etc

dansku commented 3 years ago

I am working on it! About the rewards restriction, would it be via metabase?