Skip to content

initial naive structure for at_published_ts#190

Draft
adamkorynta wants to merge 3 commits into
mainfrom
feature/a2w_refactor
Draft

initial naive structure for at_published_ts#190
adamkorynta wants to merge 3 commits into
mainfrom
feature/a2w_refactor

Conversation

@adamkorynta

Copy link
Copy Markdown
Contributor

TODO:

  • determine migration script and/or data sync trigger between old table and new
  • determine backwards compatibility for old views
  • decide if the notion of "TS_TYPE" needs to be carried forward
  • determine support for ratings, notes, date-refreshed, etc

@adamkorynta adamkorynta requested review from a team and rma-rripken and removed request for a team June 26, 2026 21:26
UNION ALL
SELECT LOCATION_CODE, 12 /* Evap */, TS_CODE_EVAP FROM AT_A2W_TS_CODES_BY_LOC WHERE TS_CODE_EVAP IS NOT NULL
UNION ALL
/* TODO: figure out how to map these to parameters. */

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure you could, ..._STORE_FLOOD, etc are Location level identifiers.

That not all projects would have. Given it's different data, you might want to approach more of a star pattern (e.g. > 3rd normal)

at_publish holding the root location and possibly (maybe in the future) other meta data, and then a table for pure time series, and another for "levels".

One generally assigns the location level, not time series, those those anyways.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants