Skip to content
This repository has been archived by the owner on Feb 10, 2023. It is now read-only.

Headers are not being properly removed to do being encoded to bytes #33

Open
onlynone opened this issue Aug 24, 2018 · 0 comments
Open

Comments

@onlynone
Copy link

I believe this commit: e88e32f broke the auto-header removal in yield_rows. The issue seems to be that the column names are encoded to 'utf8' in the headers array. This means they'll be a byte string like b'hostname'. But the data columns aren't byte strings, they're unicode strings like u'hostname'. At least in python3, these two values will not compare equal, so yield_rows thinks the first row of data is not the header row and prints it out.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant