812d0918a8
### What changes were proposed in this pull request? This PR modifies an example for `expressions.Aggregator` in Javadoc and Scaladoc. The definition of `bufferEncoder` and `outputEncoder` are added. ### Why are the changes needed? To correct the example. The current example is wrong and doesn't work because `bufferEncoder` and `outputEncoder` are not defined. ### Does this PR introduce _any_ user-facing change? Yes. Before this change, the scaladoc and javadoc are like as follows. ![wrong-example-java](https://user-images.githubusercontent.com/4736016/91897528-5ebf3580-ecd5-11ea-8d7b-e846b776ebbb.png) ![wrong-example](https://user-images.githubusercontent.com/4736016/91897509-58c95480-ecd5-11ea-81a3-98774083b689.png) After this change, the docs are like as follows. ![fixed-example-java](https://user-images.githubusercontent.com/4736016/91897592-78607d00-ecd5-11ea-9e55-03fd9c9c6b54.png) ![fixed-example](https://user-images.githubusercontent.com/4736016/91897609-7c8c9a80-ecd5-11ea-837e-9dbcada6cd53.png) ### How was this patch tested? Build with `build/sbt unidoc` and confirmed the generated javadoc/scaladoc and got the screenshots above. Closes #29617 from sarutak/fix-aggregator-doc. Authored-by: Kousuke Saruta <sarutak@oss.nttdata.com> Signed-off-by: HyukjinKwon <gurwls223@apache.org> |
||
---|---|---|
.. | ||
benchmarks | ||
src | ||
v1.2/src | ||
v2.3/src | ||
pom.xml |