databrickslabs / splunk-integration

Databricks Add-on for Splunk
https://splunkbase.splunk.com/app/5416/
Other
26 stars 18 forks source link

Databricks Splunk Integration v1.2.0 Code Chnges. #38

Closed madhavpandya-cds closed 1 year ago

madhavpandya-cds commented 1 year ago
codecov[bot] commented 1 year ago

Codecov Report

Merging #38 (c0e907e) into master (c59ef74) will decrease coverage by 3.69%. The diff coverage is 79.20%.

@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
- Coverage   86.40%   82.71%   -3.69%     
==========================================
  Files           8        9       +1     
  Lines         618      949     +331     
==========================================
+ Hits          534      785     +251     
- Misses         84      164      +80     
Impacted Files Coverage Δ
app/bin/databricksretiredrun.py 51.61% <50.00%> (-0.18%) :arrow_down:
app/bin/databricksjob.py 95.00% <62.50%> (-5.00%) :arrow_down:
app/bin/databricks_validators.py 73.12% <67.64%> (-14.38%) :arrow_down:
app/bin/databricksrun.py 95.83% <70.00%> (-4.17%) :arrow_down:
app/bin/databricksquery.py 79.59% <71.42%> (-2.13%) :arrow_down:
app/bin/databricks_common_utils.py 88.53% <82.30%> (+1.45%) :arrow_up:
app/bin/databricks_custom_decryption.py 83.00% <83.00%> (ø)
app/bin/databricks_custom_encryption.py 86.45% <86.45%> (ø)
app/bin/databricks_com.py 87.95% <93.33%> (-0.51%) :arrow_down:
...n/ta_databricks/modalert_launch_notebook_helper.py

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

alexott commented 1 year ago

Close in favor of #41