Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve spark-agent "sanity check" error messages #50

Closed
cerveada opened this issue Jan 15, 2020 · 2 comments
Closed

Improve spark-agent "sanity check" error messages #50

cerveada opened this issue Jan 15, 2020 · 2 comments
Assignees
@wajda @cerveada
Labels
enhancement
Milestone
0.5.0

Comments

@cerveada
Copy link
Contributor

The message should give more information about what is working and what not.
@cerveada cerveada self-assigned this Jan 15, 2020
@cerveada cerveada reopened this Jan 15, 2020
wajda referenced this issue in AbsaOSS/spline Jan 15, 2020
@wajda
spline #554 Conf: Use getRequiredXXX to read required properties
7647753
wajda referenced this issue in AbsaOSS/spline Jan 16, 2020
@wajda
spline #554 Conf: Use getRequiredXXX to read required properties
38474aa
@wajda wajda reopened this Jan 17, 2020
@wajda wajda self-assigned this Jan 17, 2020
@wajda
Copy link
Contributor

wajda commented Jan 17, 2020

Using getRequiredString was a quick change but is useless due to the exception thrown from the getString

@wajda
Copy link
Contributor

wajda commented Jan 17, 2020

Fixed by AbsaOSS/commons@fec10cd#diff-88ae8be6e2391263e2c736099db9a4fdR107

@wajda wajda closed this as completed Feb 10, 2020
@wajda wajda transferred this issue from AbsaOSS/spline May 5, 2020
@wajda wajda added this to the 0.5.0 milestone May 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wajda wajda

@cerveada cerveada

Labels
enhancement
Projects
Spline
Status: Done
Milestone
0.5.0
Development

No branches or pull requests
2 participants
@wajda @cerveada