[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [udig-dev] rethrow exception in EditManagerImpl#commitTransaction

Interesting, in rollbackTransaction() https://github.com/locationtech/udig-platform/blob/master/plugins/org.locationtech.udig.project/src/org/locationtech/udig/project/internal/impl/EditManagerImpl.java#L455 the Exception is re-thrown while in commitTransaction() it isn't.

IMH the catched IOException should be re-thrown in commitTransaction() either

--
Frank


Am Di., 23. Okt. 2018 um 14:43ÂUhr schrieb Loob, Sebastian <Sebastian.Loob@xxxxxxxxx>:

Hi all,

Â

is there a particular reason, why the possible exception in method org.locationtech.udig.project.internal.impl.EditManagerImpl#commitTransaction() is not rethrown (line 406)?

Currently the surrounding application does not know whether or not the transaction was succesful.

Â

So we suggest to rethrow the exception.

Â

Sebastian



IBYKUS AG fÃr Informationstechnologie, Erfurt / HRB 108616 - D-Jena / Vorstand: Helmut C. Henkel, Dr. Lutz Richter, Eckehart Klingner
Vorsitzender des Aufsichtsrates: Dr. Helmut Vorndran
_______________________________________________
udig-dev mailing list
udig-dev@xxxxxxxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.locationtech.org/mailman/listinfo/udig-dev