helium / blockchain-etl

Blockchain follower that follows and stores the Helium blockchain
Apache License 2.0
64 stars 37 forks source link

README improvements #140

Closed jamiew closed 3 years ago

jamiew commented 3 years ago
jamiew commented 3 years ago

I'm running postgres 10. I believe 11 works too. @rawrmaan did you say you ran into issues running on postgres 12?

And do we require any particular rust minimum version?

madninja commented 3 years ago

I think we need to fix #138 and adjust the GOOGLE api key instructions?

jamiew commented 3 years ago

@madninja I pretended like that is fixed already and the key is optional

stephendrobinson commented 3 years ago

I'm currently trying to install on..

Feedback points: 1) README wasn't clear that I needed to rename .env.dev to .env (otherwise the make start failed) 2) Also that postgres seemed to need postgis installed, as per sudo apt-get install postgis (might want to add a short section on installing postgres including a mention to mapping to a spare volume as the blockchain grows - I was told it could be 200GB - hence I created a 512G drive for this)

madninja commented 3 years ago

Did https://github.com/helium/blockchain-etl/pull/141 address these? @jamiew makes his PRs confusing

stephendrobinson commented 3 years ago

141 addresses all the issues - thanks!

jamiew commented 3 years ago

Sweet. FYI I'm up to about 400GB of space utilized for postgres (version10), and that is recently VACUUM'd

Up to about 35GB for blockchain ledger data

madninja commented 3 years ago

Sweet. FYI I'm up to about 400GB of space utilized for postgres (version 10), and that is recently VACUUM'd Up to about 35GB for blockchain ledger data

Quite unrelated to this issue, but I hear ya

jamiew commented 3 years ago

@madninja README still out of date without this PR

madninja commented 3 years ago

@madninja README still out of date without this PR

does it need fixes based on the other PR? hard to track given all the noise today

madninja commented 3 years ago

no it does not.. looks good

jamiew commented 3 years ago

No, each PR is self-contained