Skip to content
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

[jdbc] Use TEXT for fields instead of VARCHAR #797

Closed
wants to merge 1 commit into from

Conversation

k1xme
Copy link
Contributor

@k1xme k1xme commented Jul 10, 2016

The original statement specify field type of VARCHAR, which might not be able to store the random-length string produced by YCSB and is also inconsistent with the schema in documentation.

@busbey
Copy link
Collaborator

busbey commented Jul 22, 2016

you should update the javadocs on the jdbc client as well, it claims that things are stored in VARCHAR still

@risdenk
Copy link
Collaborator

risdenk commented Jan 19, 2017

@busbey @k1xme - Updated this PR to fix JavaDocs. New PR #910

@risdenk risdenk closed this Jan 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants