search

MetricsDAO / harmony_dbt

DBT Project for the Harmony Blockchain
12 stars 8 forks source link

Add validators table #87

Closed eevui closed 2 years ago

eevui commented 2 years ago

Following schema and discussion at https://github.com/MetricsDAO/harmony_dbt/issues/73

Testing

root@03e8142ce188:/harmony# dbt run --select validators
Running with dbt=0.21.1
Found 35 models, 284 tests, 0 snapshots, 0 analyses, 178 macros, 0 operations, 8 seed files, 6 sources, 0 exposures

14:10:33 | Concurrency: 4 threads (target='dev')
14:10:33 |
14:10:33 | 1 of 1 START incremental model DEV.validators........................ [RUN]
14:10:53 | 1 of 1 OK created incremental model DEV.validators................... [SUCCESS 1 in 20.30s]
14:10:53 |
14:10:53 | Finished running 1 incremental model in 34.62s.

Completed successfully

Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

image

Future

Currently, makes direct reference to harmony.dev.stg_delegators. Perhaps an issue can be created to make a change when stg_delegators is commited to production.

eevui commented 2 years ago

Waiting on #86 , which also resolves making a direct reference to a table.

antonyip commented 2 years ago

Looks good overall! I only have a small issue with using.

eevui commented 2 years ago

To add, just discovered that the incremental filter doesn't work. Is this expected? I get an error saying something along the lines of "aggregate query should be used with having" or something like that.

antonyip commented 2 years ago

Made a PR into your branch - https://github.com/eevui/harmony_dbt/pull/1