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

Jump to bottom

export command creates different output for the same input #3911

Closed
elifarley opened this issue Mar 9, 2015 · 5 comments
Closed

export command creates different output for the same input #3911

elifarley opened this issue Mar 9, 2015 · 5 comments
Assignees
@Tryneus
Labels
tp:bug
Milestone
1.16.3

Comments

@elifarley
Copy link

The command below was executed on 2 different hosts:

rethinkdb export -e test.txstats --format csv --fields method,count,sum,min,avg,max,s2 -d /tmp/txstats-export

The result for the first host is:

method,count,sum,min,avg,max,s2
MyClass.myMethod,154,56334,161,365.8051948051948,1260,258.7894411965032

But for the second host we have:

b'method',b'count',b'sum',b'min',b'avg',b'max',b's2'
b'MyClass.myMethod',b'154',b'56334',b'161',b'365.8051948051948',b'1260',b'258.7894411965032'
@Tryneus
Copy link
Member

Tryneus commented Mar 9, 2015

I suspect this is a problem between the script's python2/python3 compatibility. Looking into this today.

@Tryneus Tryneus self-assigned this Mar 9, 2015
@Tryneus Tryneus added this to the 2.0 milestone Mar 9, 2015
@Tryneus Tryneus added the tp:bug label Mar 9, 2015
@Tryneus Tryneus modified the milestones: 1.16.x, 2.0 Mar 9, 2015
@elifarley
Copy link
Author

Meanwhile, this workaround will help:

sed -e "s/',b'/,/g; s/^b'//; s/'//"

@Tryneus
Copy link
Member

Tryneus commented Mar 10, 2015

The fix is up in review 2701, thanks for reporting this, @elifarley!

@Tryneus
Copy link
Member

Tryneus commented Mar 10, 2015

The fix has been approved and merged into next in commit 2c8de27, cherry-picked into v2.0.x in commit ada9308, and cherry-picked into v1.16.x in commit 3ede609.

Tested with Python 2.6, 2.7, 3.1, 3.2, 3.3, and 3.4, and verified that all CSV outputs were identical.

@AtnNn, could we do a python driver release for this?

@Tryneus Tryneus closed this as completed Mar 10, 2015
@AtnNn
Copy link
Member

AtnNn commented Mar 11, 2015

A new version of the Python driver has been released with the fix, as version 1.16.0-3

@AtnNn AtnNn modified the milestones: 1.16.x, 2.0 Mar 17, 2015
@AtnNn AtnNn modified the milestones: 1.16.3, 2.0 Mar 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Tryneus Tryneus

Labels
tp:bug
Projects
None yet
Milestone
1.16.3
Development

No branches or pull requests
3 participants
@elifarley @Tryneus @AtnNn