github / searchkick

10 Jul, 2016

2 commits

09 Jul, 2016

1 commit

02 Jul, 2016

1 commit

17 May, 2016

2 commits

16 May, 2016

1 commit

04 May, 2016

4 commits

03 May, 2016

5 commits

23 Mar, 2016

1 commit

21 Mar, 2016

1 commit

20 Mar, 2016

1 commit

12 Mar, 2016

1 commit

11 Mar, 2016

1 commit

  • 1352b808   reindexing models on an ES cluster that is totally new / empty throws the following exception: ... Browse Code » 
    irb(main):001:0> User.reindex

NoMethodError: undefined method `keys' for []:Array
	from /app/vendor/bundle/ruby/2.1.0/gems/searchkick-1.2.1/lib/searchkick/index.rb:151:in `clean_indices'
	from /app/vendor/bundle/ruby/2.1.0/gems/searchkick-1.2.1/lib/searchkick/index.rb:163:in `reindex_scope'
	from /app/vendor/bundle/ruby/2.1.0/gems/searchkick-1.2.1/lib/searchkick/model.rb:51:in `searchkick_reindex'
	from (irb):1
	from /app/vendor/bundle/ruby/2.1.0/gems/railties-4.0.4/lib/rails/commands/console.rb:90:in `start'
	from /app/vendor/bundle/ruby/2.1.0/gems/railties-4.0.4/lib/rails/commands/console.rb:9:in `start'
	from /app/vendor/bundle/ruby/2.1.0/gems/railties-4.0.4/lib/rails/commands.rb:62:in `<top (required)>'
	from bin/rails:4:in `require'
	from bin/rails:4:in `<main>'

When I looked into it, client.indices.get_alias returns a hash, not an array, so the NotFound rescue should be returning an empty hash.  This change fixes the issue.

I did NOT add a unit test because the only way to test this would be to either nuke all indexes from the local ES test cluster, or startup a totally vanilla cluster just for running the test, which is kind of out of the scope of this change.
    Bernt Habermeier
     

19 Feb, 2016

1 commit

18 Feb, 2016

7 commits

17 Feb, 2016

2 commits

15 Feb, 2016

5 commits

14 Feb, 2016

4 commits