Index document without id

[sevagh]

Hello,
I've been digging through the docs (and I'm not too familiar with elasticsearch) - apologies if this is a trivial question.
It seems that it should be possible to index a document without specifying the ID, but the only examples I can find require an Id: https://github.com/elastic-rs/elastic/blob/master/src/elastic/examples/index.rs#L50
Is there a way to put a document without an id?

[KodrAus]

Hi @sevagh!

Yeh this is a bit of an oversight, the high-level document_index method requires an id, but it doesn't technically need one. I think I'll remove that parameter in the next breaking release. I have some grand plans to refactor those methods so we can infer the id and index fields from your document, but in the meantime you can send an index request manually using an IndexRequest:

use elastic::prelude::*;

#[derive(ElasticType, Serialize, Deserialize)]
struct MyType { ... }

// Serialize the document as json
let body = serde_json::to_string(&doc)?;

// Send an `IndexRequest` manually, using the index and type
// The response is then deserialized as an `IndexResponse`
let response = client
    .request(IndexRequest::for_index_ty("some-index", MyType::name(), body))
    .send()?
    .into_response::<IndexResponse>()?;

let indexed_id = response.id();

[KodrAus]

Alrighty, in the next breaking version of elastic, which will be 0.21.0, you'll be able to call document_index without having to supply an id:

client.document_index(index("some-index", doc)).send()?;

[hack3p]

This solution not worked for elastic >= 6.x #308 (comment)

thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Api(Other({"reason": String("Rejecting mapping update to [idx] as the final mapping would have more than 1 type: [mytype, body]"), "root_cause": Array([Object({"reason": String("Rejecting mapping update to [idx] as the final mapping would have more than 1 type: [mytype, body]"), "type": String("illegal_argument_exception")})]), "type": String("illegal_argument_exception")}))', libcore/result.rs:945:5

Do you plan to support elastic >= 6.x ?

[KodrAus]

Hi @hack3p. Yes, we plan to support 6.x eventually, it's tracked in #277.

I need to find some time to sit down and polish up and push out the current set of breaking changes (all that's left in a nicer bulk API) and then look at 6.x

[KodrAus]

We've got a prerelease build for 0.21.0 up on crates.io now that lets you index documents without specifying the type and id inline (we infer them if we can from the document though) so I'll go ahead and close this one. But please feel free to re-open or create new issues if anything doesn't behave the way you'd expect!