Support for edismax

[sunspot]

Support for edismax

[alindeman]

I like the idea of edismax as a configuration option at least. But I don't know all of the nuances yet. @nz, etc.. thoughts?

[vamsee]

Hi, any updates on adding edismax support to Sunspot?

[khaled]

@jashkenas did that monkey patch end up working out?

[vamsee]

@khaled it works. Thanks, @jashkenas.

[jashkenas]

Yes -- the monkey patch is currently working for us.

[nz]

Edismax is great. We should be using it. I am getting close to doing a Sunspot sprint that drops all vestiges of Solr 1.4 support in favor of Solr 3 and 4, at which time this would be a prime candidate for inclusion.

[inspire22]

Any luck with this?

[tadast]

Any plans to proceed with this?
Is there any reason why #371 doesn't get merged in? If there is anything I could help with, please let me know @nz.

[zquestz]

We are also using the monkey patch with great success. When do you guy's plan to merge this to the main repo? I like my paren support.

[heaven]

A little improved version:

Sunspot::Query::OriginalDismax = Sunspot::Query::Dismax

class Sunspot::Query::PatchedDismax < Sunspot::Query::OriginalDismax
  def to_params
    super.merge(:defType => "edismax")
  end

  def to_subquery
    super.sub("{!dismax", "{!edismax")
  end
end

Sunspot::Query.send :remove_const, :Dismax
Sunspot::Query::Dismax = Sunspot::Query::PatchedDismax

[heaven]

Also there is a difference in query parsing, see http://lucene.472066.n3.nabble.com/Edismax-vs-Dismax-td4083812.html

[rywall]

Closed by #487.