Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Diff between spec and DDL for observation_period table #191

Closed
pavgra opened this issue Jun 18, 2018 · 4 comments
Closed

Diff between spec and DDL for observation_period table #191

pavgra opened this issue Jun 18, 2018 · 4 comments

Comments

@pavgra
Copy link

pavgra commented Jun 18, 2018

DDL:
image
(the same w/ Postgres)

Spec:
image

@clairblacketer
Copy link
Contributor

Thanks @pavgra for finding this, the specification is correct in this case and OBSERVATION_PERIOD_START_DATETIME/OBSERVATION_PERIOD_END_DATETIME should not be in the DDL. This is fixed in the most recent release.

@pavgra
Copy link
Author

pavgra commented Jun 18, 2018

@clairblacketer , could you tell how that most recent release of the 5.2 can be found?

@clairblacketer
Copy link
Contributor

HI @pavgra I just released CDM v5.2.2 as we decided to continue supporting older versions of the CDM when issues like this come up.

@pavgra
Copy link
Author

pavgra commented Jun 22, 2018

@clairblacketer , that's great, thank you!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants