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
Error in the documentation: Field accessor post-aggregator #1584
Comments
fix for the issue: apache#1584
@MarConSchneid do you think you can submit a PR for this fix? |
Sorry, for letting you wait, but I'm currently on parental leave. Let me know if I did it correct. (first time for me to contribute to a big project). |
@MarConSchneid it looks like you submitted the PR to your own repository instead of druid-io. If it's too much trouble the re-submit, I can merge your changes directly from your repo. We just need you to sign our CLA first here, http://druid.io/community/cla.html Thanks! |
@xvrl I've signed the CLA. And sorry for the trouble. |
fix for the issue: apache#1584
Issue open based on the discussion: https://groups.google.com/forum/#!topic/druid-user/zUpDX1H2_Nw
The general example for the Field accessor post-aggregator (http://druid.io/docs/latest/querying/post-aggregations.html) has a wrong syntax:
{ "type" : "fieldAccess", "fieldName" : <aggregator_name> }
and further down the page in the example you can find this code example (which works):
{ "type" : "fieldAccess", "name" : "tot", "fieldName" : "tot" }
With best regards
Martin Schneider
The text was updated successfully, but these errors were encountered: