stratumn / go-core

Stratumn's core utilities to create decentralized applications and networks
Apache License 2.0
0 stars 0 forks source link

postgres: add schemas #460

Closed t-bast closed 6 years ago

t-bast commented 6 years ago

Split the table in two different schemas. Trace should do the same: its tables should be in a "trace" schema.

We can then configure roles for schema access. I think this shouldn't be done inside go-core but rather as an OPS script on the DB itself. Here is what I think we should do in such a script:

Does that make sense?


This change is Reviewable

codecov[bot] commented 6 years ago

Codecov Report

Merging #460 into master will decrease coverage by 0.12%. The diff coverage is 6.97%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #460      +/-   ##
==========================================
- Coverage   72.81%   72.68%   -0.13%     
==========================================
  Files          86       86              
  Lines        4969     4976       +7     
==========================================
- Hits         3618     3617       -1     
- Misses       1015     1024       +9     
+ Partials      336      335       -1
Impacted Files Coverage Δ
postgresstore/cmd.go 0% <0%> (ø) :arrow_up:
postgresstore/postgresstore.go 76.25% <0%> (+1.55%) :arrow_up:
postgresstore/stmts.go 98.29% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 7732795...86a1573. Read the comment docs.