loquisgon commented on a change in pull request #11123:
URL: https://github.com/apache/druid/pull/11123#discussion_r622658383
##########
File path:
server/src/main/java/org/apache/druid/segment/realtime/appenderator/AppenderatorImpl.java
##########
@@ -813,6 +851,24 @@ private DataSegment mergeAndPush(
Closer closer = Closer.create();
try {
for (FireHydrant fireHydrant : sink) {
+
+ // if batch, swap/persist did not memory map the incremental index,
we need it mapped now:
+ if (!isRealTime()) {
+ // sanity:
+ if (fireHydrant.getPersistedFile() == null) {
+ throw new ISE("Persisted file for batch hydrant is null!");
Review comment:
Added hydrant (toString), a little more but not a lot more.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
