Replies: 1 comment
-
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Problem description
In #489 we decided to remove
REQUIRED
mode. There were a few reasons for this:In other words
REQUIRED
mode was not strong enough to be called that way, and in many cases behaved like theBEST_EFFORT
due to limitations of Spark listener API. That why we replaced both modes with justENABLED
.But... there still are cases when fast failure is desired and is technically possible. Like, for instance, when using programmatic initialization method by explicitly calling
enableLineageTracking()
. In that case a user might logically expect the program to throw an exception if lineage tracking cannot be enabled for whatever reason. Unfortunately, after the aforementioned change the above method always returns with no error, and there is no way for a user to enforce the exception throwing, or to verify the agent initialization status.Questions to the community
a) should the
enableLineageTracking()
always throw in case of failure, orb) should this behavior be optional, and be controlled by, say a boolean method parameter (e.g.
enableLineageTracking(..., silentOnError = true)
to swallow errors, otherwise it would throw by default)c) should we add another method (say
isLineageTrackingEnabled()
) to let a user to check the agent statusBeta Was this translation helpful? Give feedback.
All reactions