sandeep318kumar commented on code in PR #3825:
URL: https://github.com/apache/ambari/pull/3825#discussion_r1766266498
##########
ambari-server/src/main/java/org/apache/ambari/server/controller/metrics/timeline/cache/TimelineMetricCacheEntryFactory.java:
##########
@@ -319,4 +322,24 @@ private long getMillisecondsTime(long time) {
return time;
}
}
+ @Override
+ public void delete(TimelineAppMetricCacheKey key) throws
CacheWritingException {
+ // no need to implement.
+ }
+
+ @Override
+ public Map<TimelineAppMetricCacheKey, TimelineMetricsCacheValue>
loadAll(Iterable<? extends TimelineAppMetricCacheKey> keys) throws
BulkCacheLoadingException, Exception {
+// no need to implement.
+ return null;
+ }
+
+ @Override
+ public void writeAll(Iterable<? extends Map.Entry<? extends
TimelineAppMetricCacheKey, ? extends TimelineMetricsCacheValue>> entries)
throws BulkCacheWritingException, Exception {
+// no need to implement.
+ }
+
+ @Override
+ public void deleteAll(Iterable<? extends TimelineAppMetricCacheKey> keys)
throws BulkCacheWritingException, Exception {
+// no need to implement.
+ }
Review Comment:
@JiaLiangC
We are implementing CacheLoaderWriter interface, that's why we need to
provide implementation of all its methods.
--
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]
