vanzin commented on a change in pull request #24616: [SPARK-27726] [Core] Fix performance of ElementTrackingStore deletes when using InMemoryStore under high loads URL: https://github.com/apache/spark/pull/24616#discussion_r285751231
########## File path: common/kvstore/src/main/java/org/apache/spark/util/kvstore/InMemoryStore.java ########## @@ -248,16 +337,16 @@ private int compare(T e1, T e2, KVTypeInfo.Accessor getter) { diff = compare(e1, natural, natural.get(e2)); } return diff; - } catch (Exception e) { - throw Throwables.propagate(e); + } catch (ReflectiveOperationException e) { + throw new RuntimeException(e); } } private int compare(T e1, KVTypeInfo.Accessor getter, Object v2) { try { return asKey(getter.get(e1)).compareTo(asKey(v2)); Review comment: Did you try to use the `getPredicate` stuff in this class too? Seems like it could save some computation. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org