Update java implementation status#364
Conversation
codeboten
left a comment
There was a problem hiding this comment.
What's the best way to review this? Should someone in the Java implementation review the status?
MrAlias
left a comment
There was a problem hiding this comment.
As a configuration SIG member I can say the syntax looks good. I'd like to have another Java SIG member give this a review for content before merging.
|
Will reach out to the java SIG. TBH, its a tough ask because a thorough review requires the reviewer to go through the implementation with a fine tooth comb like I needed to do to open this PR. I favor rubber stamps for this type of thing, and followup fixes if / when discovered. |
|
Ah good catch @zeitlinger! I think I made an error merging conflicts from #362 since I called out support for specific properties while indicating that ConsoleMetricExporter itself was unsupported. Fixed in #380. |
Did a thorough pass on the java implementation and updated current implementation status.