Github user cestella commented on the issue:

    https://github.com/apache/metron/pull/1012
  
    Ok, this looks great, one request though.  It appears that, at least for 
the profiler, it is required that objects used in Stellar be Java serializable 
if they aren't listed in that exception list.
    
    We ensure that the objects are kryo serializable 
[here](https://github.com/apache/metron/blob/91a017b6dcefc250bfd67cecf9803cb59015d213/metron-stellar/stellar-common/src/main/java/org/apache/metron/stellar/common/utils/StellarProcessorUtils.java#L64)
  Can we also make a blurb that they're java serializable?  I know that 
OnlineStatisticsProvider will not be java serializable due to its dependence on 
t-digest, so maybe make that check optional but on by default?



---

Reply via email to