Skip to content

Commit

Permalink
Revert "Temporarily revert rethinkdb upgrade"
Browse files Browse the repository at this point in the history
Turns out that despite the fact that we were running rethinkdb version 1.16,
we still had indexes which were created in 1.13.0. There is not a direct
migration path from 1.13.0 indexes straight to 2.2.0. Instead I had to migrate
first to 2.0.5 (Note 2.0.0 SHOULD have worked but there was a bug
rethinkdb/rethinkdb#4775 which we seem to have been
subject to). Once I was on 2.0.5 I was able to rebuild the indexes. From there
I was able to bring us back to 2.2.0 and things worked again.

For the curious, here's the error we were seeing:

    2016-04-15T05:42:01.393253255 0.010778s notice: Running rethinkdb 2.2.0~0trusty (GCC 4.8.2)...
    2016-04-15T05:42:01.407714269 0.025237s notice: Running on Linux 4.4.0-x86_64-linode63 x86_64
    2016-04-15T05:42:01.407772464 0.025290s notice: Loading data from directory /var/lib/rethinkdb/instance1/data
    2016-04-15T05:42:01.419155695 0.036673s notice: Migrating cluster metadata to v2.2
    2016-04-15T05:42:01.875603265 0.493121s error: Found a secondary index from unsupported RethinkDB version 1.13.  You can migrate this secondary index using RethinkDB 2.0.
  • Loading branch information
captbaritone committed Apr 15, 2016
1 parent b008819 commit ad956f4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion requirements.in
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
raven[flask]
rethinkdb==1.14.0-0
rethinkdb==2.2.0.post6
Flask==0.9
requests>=1.2.3,<2.0.0
lxml>=3.2.1,<4.0.0
Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ python-slugify==0.0.6
PyYAML==3.11
raven[flask]==5.12.0
requests==1.2.3
rethinkdb==1.14.0-0
rethinkdb==2.2.0.post6
six==1.10.0 # via html5lib, python-dateutil
termcolor==1.1.0
tox==2.3.1
Expand Down

2 comments on commit ad956f4

@divad12
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for figuring this out!

@thejsj
Copy link

@thejsj thejsj commented on ad956f4 Apr 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow. That's an interesting one.

Please sign in to comment.