Skip to content
This repository has been archived by the owner on Jan 19, 2022. It is now read-only.

Added index and modified datatype for sec and nsec to better reflex 32 bits content #45

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

y-trudeau
Copy link

The index is needed for the purge-qan-data script. The sec and nsec column contain 32 bits unsigned values, why use 64 bits int?

@CLAassistant
Copy link

CLAassistant commented Jan 10, 2018

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@delgod
Copy link
Contributor

delgod commented Jan 11, 2018

should we update old systems? or they can use old schema?

Copy link
Contributor

@askomorokhov askomorokhov left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@rnovikovP rnovikovP added 2.x 1.x and removed 2.x labels Mar 21, 2019
@delgod delgod removed their request for review June 6, 2019 11:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants