Event name existence check SQLException catch is too general

CenturyLinkCloud / mdw

https://centurylinkcloud.github.io/mdw/

MIT License

46 stars 10 forks source link

Event name existence check SQLException catch is too general #760

Closed donaldoakes closed 4 years ago

donaldoakes commented 4 years ago

In EngineDataAccessDB.recordEventArrive() and EngineDataAccessDB.recordEventWait(), Instead of assuming that any SQLException is due to an already-existing row with the same EVENT_NAME, we should specifically catch SQLIntegrityConstraintViolationException and let other SQLExceptions propagate.