-
Notifications
You must be signed in to change notification settings - Fork 503
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
[DOC]Add new dot expander processor doc #5631
Conversation
Signed-off-by: Melissa Vagi <[email protected]>
@msfroh Do you have availability to review this PR? I don't want to dump all of the ingest processor reviews on you, so if there are other devs I can tag to share the review load, please let me know. Thank you! |
Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added examples of path parameter and field name conflicts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vagimeli Please see my comments and changes and let me know if you have any questions. Thanks!
``` | ||
{% include copy-curl.html %} | ||
|
||
The response shows that the values were merged into an array: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"indicates" instead of "shows"? (optional suggestion)
} | ||
``` | ||
|
||
If there is a field name with a same name but a different path field needs to be renamed. For example, the following simulate call returns a parse exception: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"If a field contains the same name but a different path, then the field needs to be renamed"?
``` | ||
{% include copy-curl.html %} | ||
|
||
Now simulate the pipeline: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Now you can simulate the pipeline"?
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]>
Signed-off-by: Melissa Vagi <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vagimeli Just one comment. Great job 😄
} | ||
``` | ||
|
||
If a field contains the same name but a different path, then the field needs to be renamed. For example, the following simulate call returns a parse exception: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should "simulate" be in code font (my brain wants to incorrectly change it to "simulated")?
Thanks @natebower! I changed to |
Signed-off-by: Melissa Vagi <[email protected]>
* Add new dot expander processor doc Signed-off-by: Melissa Vagi <[email protected]> * Draft content for tech review Signed-off-by: Melissa Vagi <[email protected]> * Address tech review feedback Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Naarcha-AWS <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Address doc review feedback Signed-off-by: Melissa Vagi <[email protected]> * Edit line 227 Signed-off-by: Melissa Vagi <[email protected]> * Edit line 227 Signed-off-by: Melissa Vagi <[email protected]> * Edit line 227 Signed-off-by: Melissa Vagi <[email protected]> * Address doc review comments Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: kolchfa-aws <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Added path parameter and field name conflicts sections Signed-off-by: Fanit Kolchina <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Update _ingest-pipelines/processors/dot-expander.md Co-authored-by: Nathan Bower <[email protected]> Signed-off-by: Melissa Vagi <[email protected]> * Address editorial review feedback Signed-off-by: Melissa Vagi <[email protected]> * Update dot-expander.md Signed-off-by: Melissa Vagi <[email protected]> --------- Signed-off-by: Melissa Vagi <[email protected]> Signed-off-by: Fanit Kolchina <[email protected]> Co-authored-by: Naarcha-AWS <[email protected]> Co-authored-by: kolchfa-aws <[email protected]> Co-authored-by: Fanit Kolchina <[email protected]> Co-authored-by: Nathan Bower <[email protected]> (cherry picked from commit 8225f0b) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Add new dot expander processor doc * Draft content for tech review * Address tech review feedback * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Address doc review feedback * Edit line 227 * Edit line 227 * Edit line 227 * Address doc review comments * Update _ingest-pipelines/processors/dot-expander.md * Added path parameter and field name conflicts sections * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Update _ingest-pipelines/processors/dot-expander.md * Address editorial review feedback * Update dot-expander.md --------- (cherry picked from commit 8225f0b) Signed-off-by: Melissa Vagi <[email protected]> Signed-off-by: Fanit Kolchina <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Naarcha-AWS <[email protected]> Co-authored-by: kolchfa-aws <[email protected]> Co-authored-by: Fanit Kolchina <[email protected]> Co-authored-by: Nathan Bower <[email protected]>
Description
Adds new dot expander processor documentation to address content gap
Issues Resolved
#4647
Checklist
For more information on following Developer Certificate of Origin and signing off your commits, please check here.