Philippus / elastic4s

🔍 Elasticsearch Scala Client - Reactive, Non Blocking, Type Safe, HTTP Client
Apache License 2.0
1.64k stars 695 forks source link

Update id ... in "idx/type" should accept a map as well #68

Closed hura closed 10 years ago

hura commented 10 years ago

Works:

update id 5 in "scifi/startrek"

Doesn't work:

update id 5 in "scifi" -> "startrek"
sksamuel commented 10 years ago

You sure, because there's this test

  it should "accept tuple for in" in {
    val req = update id 65 in "places" -> "cities"
    assert(req.build.index() === "places")
    assert(req.build.`type`() === "cities")
  }
hura commented 10 years ago

Ah sorry about that. I'm still on the latest release version from July and never checked the online sources. And it's late :)

sksamuel commented 10 years ago

:)

On 7 January 2014 01:00, hura notifications@github.com wrote:

Ah sorry about that. I'm still on the latest release version from July and never checked the online sources. And it's late :)

— Reply to this email directly or view it on GitHubhttps://github.com/sksamuel/elastic4s/issues/68#issuecomment-31704232 .