search

databrickslabs / splunk-integration

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

Update Splunk DB Connect guide with new JDBC information #45

Closed metrocavich closed 1 year ago

metrocavich commented 1 year ago

fixed JDBC url to not point at ODBC

fixed class name to be correct with newest version

Fixed url pattern

codecov[bot] commented 1 year ago

Codecov Report

Merging #45 (564ea86) into master (bde8443) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   82.69%   82.69%           
=======================================
  Files           9        9           
  Lines         780      780           
=======================================
  Hits          645      645           
  Misses        135      135
metrocavich commented 1 year ago

lgtm, how was it tested?

I met with a customer who was having issues with the "databricks_spark_sql" driver showing up. We modified the driver class configuration and restarted the splunk service and immediately had it show up.

The old URL clearly was linking to an ODBC driver. The URL pattern in theory does not need to be updated, as the documentation specifically has you override the pattern. However, it is the pattern needed to connect to the Databricks clusters.