New issue

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

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

Already on GitHub? Sign in to your account

update doc and add tests for git issue 2 #3

Merged
merged 1 commit into from Mar 24, 2015

Conversation

Projects
None yet
2 participants
@lharey
Contributor

lharey commented Mar 24, 2015

Hi

As discussed in out emails, I've updated the documentation to state the importance of order of components and added tests to prove that component all works ok if Timestamp is loaded after EncodedColumn.

Let me know what you think

Lisa

wreis added a commit that referenced this pull request Mar 24, 2015

Merge pull request #3 from lharey/master
update doc and add tests for git issue #2

@wreis wreis merged commit e000c26 into wreis:master Mar 24, 2015

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