elasticsearch icon indicating copy to clipboard operation
elasticsearch copied to clipboard

Boolean field to support ignore_malformed

Open javanna opened this issue 2 years ago • 10 comments

The ignore_malformed option is supported by a few of our field types, mainly numerics and dates, to still accept the document when such fields hold malformed values. See https://www.elastic.co/guide/en/elasticsearch/reference/current/ignore-malformed.html. Also ignore_malformed is an index level setting that gets applied to all field types that support it.

The boolean field type does not support ignore_malformed but it should, so that if a boolean field cannot be parsed as a boolean, the document can still be accepted.

javanna avatar Aug 23 '22 13:08 javanna

Pinging @elastic/es-search (Team:Search)

elasticsearchmachine avatar Aug 23 '22 13:08 elasticsearchmachine

Hello @javanna, I would like to start my contribution to elasticsearch from this issue. Can I pick this up??

cc: @elasticsearchmachine

devnamrits avatar Aug 24 '22 04:08 devnamrits

Yes @devnamrits thanks for offering your help, feel free to open a PR around this. let us know if you need guidance.

javanna avatar Aug 26 '22 07:08 javanna

Hi @javanna, Please assign this to me. I've built the project in my local and currently I'm going through the documentation. Also, it would be great if you can streamline things from the code point of view for this issue.

devnamrits avatar Aug 27 '22 19:08 devnamrits

Hi @javanna , This is also my first contribution ,so do i need linux for contribution currently i am using windows

akshaybhadange avatar Sep 14 '22 06:09 akshaybhadange

heya, I can't assign the issue to external contributors, but @devnamrits said he was interested a couple of weeks ago hence I would assume he will open a PR. @akshaybhadange I think it would be best to find another issue to work on to avoid overlap. here are some guidelines on contributing to the project: https://github.com/elastic/elasticsearch/blob/main/CONTRIBUTING.md#contributing-code-and-documentation-changes

javanna avatar Sep 14 '22 14:09 javanna

@javanna I've been working on it. I went through the test cases where you are testing the ignore_malformed. Can you please tell me where within code, ignore_malformed setting is enabled for currently supported data types. It would be really help as then I can deliver this fast if I get to know how { ignore_malformed: true } is being enabled for currently supporting data types.

devnamrits avatar Sep 14 '22 17:09 devnamrits

@javanna Raised PR for this issue. #90122

devnamrits avatar Sep 20 '22 03:09 devnamrits

@devnamrits thanks, I've seen it, I will review it. It will take me a few days to get to it though, thanks for your patience.

javanna avatar Sep 21 '22 07:09 javanna

Hi @javanna Thank you for your kind reply. Let me know if changes are required. Also it would be great, meanwhile, if you can assign me some other issue. Would love to pick them up.

devnamrits avatar Sep 21 '22 08:09 devnamrits

Hello, would you mind if I worked on this issue as well? This will be my first open source contribution of any sort; I am (hopefully-understandably) a little nervous about contributions, but this issue and the elasticsearch project look interesting to me.

jem-n-em avatar Sep 30 '22 22:09 jem-n-em

Hi @jem-n-em I've already raised PR for this. @javanna is reviewing. So @javanna can tell whether you should pick or not.

devnamrits avatar Oct 01 '22 08:10 devnamrits

Hi @jem-n-em I've already raised PR for this. @javanna is reviewing. So @javanna can tell whether you should pick or not.

oh I'm sorry, didn't see the conversation above, thank you anyways.

jem-n-em avatar Oct 01 '22 21:10 jem-n-em

Hi @javanna , I want to contribute in open source and I don't know from where to start so could you please help me to get started with open source and give any beginner friendly issues which I can solve.

khunt-miral avatar Dec 17 '22 15:12 khunt-miral