jihoonson commented on a change in pull request #9530: DruidSegmentReader
should work if timestamp is specified as a dimension
URL: https://github.com/apache/druid/pull/9530#discussion_r398089315
##########
File path:
indexing-service/src/main/java/org/apache/druid/indexing/input/DruidSegmentReader.java
##########
@@ -270,6 +273,17 @@ public boolean hasNext()
theEvent.put(metric, value);
}
}
+
+ /*
+ * Timestamp is added last because we expect that the time column will
always be a date time object.
+ * If it is added earlier, it can be overwritten by metrics or
dimenstions with the same name.
+ *
+ * If a user names a metric or dimension `__time` it will be
overwritten. This case should be rare since
Review comment:
Ah good point. Now I think we need some schema validation for ingestion
which could probably be done in `DataSchema`. But this would be a larger issue
than the bug this PR fixes, and I'm ok with adding it later.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
