-
Notifications
You must be signed in to change notification settings - Fork 9
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
[RFE] More metadata fields #47
Comments
IMHO having consistency across our metadata would be ideal. Not sure if there is additional benefit adding the metadata across indexes. |
+1. We should look for options to keep metadata minimal. Replicating it across indexes is not necessary. |
If it is information we either are not retaining or is difficult to retrieve from what we do capture, then yes I see the value in capturing. |
I think we have two things regarding metadata here
cc: @chentex |
Appending slack thread link, that captures the overall idea. |
One thing to consider after more thought and exploration here... All of our CPT runs use |
Add more fields to metadata collection, the ones at https://github.com/cloud-bulldozer/e2e-benchmarking/blob/bf5ac71356e1f128f35cb231ad67e39729837345/utils/index.sh#L183C1-L190C53 can be potentially included
Im not sure about the actual benefits of indexing them along with the tests results though. wdyt @jtaleric @dry923 @afcollins , etc?
cc: @paigerube14 @vishnuchalla
The text was updated successfully, but these errors were encountered: