-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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
introduce the JSON Processor #20128
introduce the JSON Processor #20128
Conversation
1997087
to
2c034ea
Compare
-------------------------------------------------- | ||
{ | ||
"json": { | ||
"field": "my_json_string" |
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.
Could it be a real JSON string in doc?
b1775b3
to
911ae8f
Compare
@martijnvg mind reviewing this one as well? |
0f18d35
to
80f2b9c
Compare
/** | ||
*/ | ||
public class IngestDocumentMatcher { | ||
public static void assertIngestDocument(Object a, Object b) { |
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.
maybe remove the empty jdocs at the class level and add actual jdoc on the method itself?
LGTM, assuming build is happy. Left one small comment. |
thanks! I will wait for the other PR to merge in so that I can remove this |
80f2b9c
to
ac38a7c
Compare
Closes #20052