Elasticsearch: Support nested property names in field auto completion #61165
Replies: 11 comments
-
|
Doesn't PR #4527 fix this? Thought of connecting it here. |
Beta Was this translation helpful? Give feedback.
-
|
@arcolife yes, that PR does that. not sure what your referring to about tests / timestamp. the reason it is not merged is that it has no unit tests. |
Beta Was this translation helpful? Give feedback.
-
|
@arcolife, do you have a plan for adding unit tests here? |
Beta Was this translation helpful? Give feedback.
-
|
@portante I had added unit tests later on. Tracked through #4527 and finally through #4694 (comment) the autocomplete feature was implemented in the same PR. #4694 (comment) the reason I commented above refering #4527 was because I had earlier opened up another issue #4526 which was similar to this one right here. |
Beta Was this translation helpful? Give feedback.
-
|
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Beta Was this translation helpful? Give feedback.
-
|
Sorry, stale bot was wrongly configured (should only have acted on PRs) |
Beta Was this translation helpful? Give feedback.
-
|
Hey @torkelo, can you add some details for this, is this still relevant today? |
Beta Was this translation helpful? Give feedback.
-
|
Not sure, is it already implemented? |
Beta Was this translation helpful? Give feedback.
-
|
@Elfo404 can you take a look if this is still an issue seem like there were some PRs already merged for this so it's probably already fixed. |
Beta Was this translation helpful? Give feedback.
-
|
Took some time to look at this and better understand the context. Apparently it received some interest over time (although not a lot compared to other feature requests). I'm leaving the "needs investigation" label as I'm not how tricky it would be to implement and I'm not sure about the priority we should assign to this. |
Beta Was this translation helpful? Give feedback.
-
|
Hello, as you may have heard, we are transitioning away from using discussions to discuss feature requests. We are migrating this discussion to an issue and closing the discussion. The issue is #83006. Feel free to continue the discussion around this there. Thank you! |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
All reactions