adarshsanjeev commented on PR #13793:
URL: https://github.com/apache/druid/pull/13793#issuecomment-1449386515
After some discussion with Paul about the correct approach, we determined
that during compaction time rollup, the LATEST_BY aggregator should fetch the
correct value from the object value. Hence, LATEST BY should be okay as a
workaround for LATEST, as that still does not function correctly.
Since there are no large issues blocking this PR, I will work on addressing
any minor comments and getting this merged since currently it could produce
incorrect result without this validation..
I will also try to resolve the issues with LATEST by updating the reference
to time column, but this might take a little more effort to come up with the
best approach. I will track any updates to this in the github issue.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
