Skip to content

Allow _meta in mappings on a field level #2857

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
rore opened this issue Apr 4, 2013 · 5 comments
Closed

Allow _meta in mappings on a field level #2857

rore opened this issue Apr 4, 2013 · 5 comments
Labels

Comments

@rore
Copy link

rore commented Apr 4, 2013

A mapping can have a _meta element - but it only keeps it if declared on the top level.
When using the _meta to annotate a mapping, or keeping custom configuration on it, or using it for some binding - a lot of times besides the top level you also need more custom data on a field level. So you need the mapping to keep also _meta nodes defined under fields.

@clintongormley
Copy link
Contributor

I'd rather avoid an explosion of meta fields - easy enough to store this in the type-level meta. Closing

@awick
Copy link

awick commented Dec 5, 2014

Unfortunately this means you have to keep 2 things in sync, every time you add a field you have to modify the type meta. If you have multiple apps that can add fields you have consistency issues, type meta isn't versioned or "partial" updatable. Example: To add a field you only have to put the mapping of the 1 field and it will be merged, and optionally ignored if already there. Now to update meta you'll have to fetch, change, put.

So it isn't actually "easy" to use type meta.

@mindbits
Copy link

mindbits commented Feb 4, 2015

+1

@zjsun
Copy link

zjsun commented Dec 18, 2017

Strongly requesting field-level meta, please!

@awick
Copy link

awick commented Jan 21, 2020

Yeah!! Looks like we are finally getting this in #49419

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants