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

rethinkdb-dump fails if there are no tables #1904

Closed
AtnNn opened this issue Jan 24, 2014 · 5 comments
Closed

rethinkdb-dump fails if there are no tables #1904

AtnNn opened this issue Jan 24, 2014 · 5 comments
Assignees
Milestone

Comments

@AtnNn
Copy link
Member

AtnNn commented Jan 24, 2014

With a server that has no tables:

$ rethinkdb dump -c localhost:28018
NOTE: 'rethinkdb-dump' only dumps data and does *not* dump secondary indexes or
 cluster metadata.  You will need to recreate your secondary indexes and cluster
 setup yourself after you run 'rethinkdb-restore'.
Exporting to directory...
[========================================] 100% 
Failed to move temporary directory to output directory (/tmp/tmpEl4uw_/rethinkdb_dump_2014-01-24T13:47:13): No such file or directory
Error: rethinkdb-export failed

If I create a table, the dump succeeds.

This is possibly the same bug as http://stackoverflow.com/questions/21311887/how-can-i-run-rethinkdb-dump-export-in-rethinkdb-docker

Assigning to @Tryneus

@ghost ghost assigned Tryneus Jan 24, 2014
@coffeemug
Copy link
Contributor

We should also check export and such to make sure there are no other edge cases here (sorry for stating the obvious).

@Tryneus
Copy link
Member

Tryneus commented Jan 25, 2014

Dump uses export, this is actually an error happening in rethinkdb-export. Fix is pretty simple, will have it up soon.

@Tryneus
Copy link
Member

Tryneus commented Jan 25, 2014

Fix is up in review 1154.

@Tryneus
Copy link
Member

Tryneus commented Jan 27, 2014

The fix has been approved and merged into next in commit fb4e9df, and cherry-picked into v1.11.x in commit 79dd40c. Will be in release 1.11.4 and/or 1.12.

@Tryneus Tryneus closed this as completed Jan 27, 2014
@AtnNn
Copy link
Member Author

AtnNn commented Jan 28, 2014

I have released a new version of the Python driver with this change (version 1.11.0-2), it should be available on pypi shortly.

@AtnNn AtnNn modified the milestones: 1.12, 1.11.x Mar 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants