JeffBolle opened a new pull request, #11551: URL: https://github.com/apache/pinot/pull/11551
Fixes https://github.com/apache/pinot/issues/11530 Appears to duplicate https://github.com/apache/pinot/pull/11544 however in testing @navina 's PR, I noticed that my code was still failing validation when testing it with an actual docker build. I looked at it again and saw that the `streamConfig.getDecoderProperties()` call was returning a simple map of just the decoder properties, not what the original author had assumed. For the test data, the returned map is: ``` {protoClassName=test, descriptorFile=file://test/} ``` When examining the tests, I saw that they only validated the failure cases and not a full passing case. Label: `bugfix` @navina @xiangfu0 / @Jackie-Jiang -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
