-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56417/
-----------------------------------------------------------
Review request for atlas and David Kantor.
Bugs: ATLAS-1535
https://issues.apache.org/jira/browse/ATLAS-1535
Repository: atlas
Description
-------
I've added a new filter to consistently clean up stale transactions when
processing http requests. I removed the old logic, which was only used by two
service classes. The new filter is used everywhere.
Diffs
-----
webapp/src/main/java/org/apache/atlas/web/filters/StaleTransactionCleanupFilter.java
PRE-CREATION
webapp/src/main/java/org/apache/atlas/web/listeners/GuiceServletConfig.java
d0437fc54087e6803ead83af59b85e9f6df333ad
webapp/src/main/java/org/apache/atlas/web/resources/BaseService.java
fb77b11e04c875a3d443dfb49be50cb818bed441
webapp/src/test/java/org/apache/atlas/web/resources/TaxonomyServiceTest.java
e1734e465ad553d95735b466f4dfe0fa8ad061b8
Diff: https://reviews.apache.org/r/56417/diff/
Testing
-------
Ran webapp tests. Now, we're only left with the following failures:
QuickStartIT.runQuickStart:44 » AtlasService Metadata service API
org.apache.a...
QuickStartV2IT.runQuickStart:47 » AtlasService Metadata service API
org.apache...
NotificationHookConsumerIT.testUpdatePartialUpdatingQualifiedName:170
expected:<0> but was:<1>
Previously, there were a number of other tests that were failing sporatically
with a NullPointerException coming from Titan. These changes fix that.
Thanks,
Jeff Hagelberg
