Skip to content

Commit afbf8f8

Browse files
committed
Statement Latency
1 parent ba4c0dc commit afbf8f8

9 files changed

Lines changed: 549 additions & 368 deletions

File tree

src/main/java/com/databricks/jdbc/dbclient/IDatabricksClient.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ public interface IDatabricksClient {
2828
* @param sessionConf session configuration
2929
* @return created session
3030
*/
31+
@DatabricksMetricsTimed
3132
ImmutableSessionInfo createSession(
3233
IDatabricksComputeResource computeResource,
3334
String catalog,
@@ -73,6 +74,7 @@ DatabricksResultSet executeStatement(
7374
* @param parentStatement statement instance if called from a statement
7475
* @return response for statement execution
7576
*/
77+
@DatabricksMetricsTimed
7678
DatabricksResultSet executeStatementAsync(
7779
String sql,
7880
IDatabricksComputeResource computeResource,
@@ -86,13 +88,15 @@ DatabricksResultSet executeStatementAsync(
8688
*
8789
* @param statementId statement which should be closed
8890
*/
91+
@DatabricksMetricsTimed
8992
void closeStatement(StatementId statementId) throws DatabricksSQLException;
9093

9194
/**
9295
* Cancels a statement in Databricks server
9396
*
9497
* @param statementId statement which should be aborted
9598
*/
99+
@DatabricksMetricsTimed
96100
void cancelStatement(StatementId statementId) throws DatabricksSQLException;
97101

98102
/**

src/main/java/com/databricks/jdbc/dbclient/IDatabricksMetadataClient.java

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,17 @@
22

33
import com.databricks.jdbc.api.impl.DatabricksResultSet;
44
import com.databricks.jdbc.api.internal.IDatabricksSession;
5+
import com.databricks.jdbc.telemetry.latency.DatabricksMetricsTimed;
56
import java.sql.SQLException;
67

78
public interface IDatabricksMetadataClient {
89

910
/** Returns information about types supported by Databricks server */
11+
@DatabricksMetricsTimed
1012
DatabricksResultSet listTypeInfo(IDatabricksSession session) throws SQLException;
1113

1214
/** Returns the list of catalogs */
15+
@DatabricksMetricsTimed
1316
DatabricksResultSet listCatalogs(IDatabricksSession session) throws SQLException;
1417

1518
/**
@@ -21,6 +24,7 @@ public interface IDatabricksMetadataClient {
2124
* absolute name)
2225
* @return a DatabricksResultSet representing list of schemas
2326
*/
27+
@DatabricksMetricsTimed
2428
DatabricksResultSet listSchemas(
2529
IDatabricksSession session, String catalog, String schemaNamePattern) throws SQLException;
2630

@@ -35,6 +39,7 @@ DatabricksResultSet listSchemas(
3539
* absolute name)
3640
* @return a DatabricksResultSet representing list of tables
3741
*/
42+
@DatabricksMetricsTimed
3843
DatabricksResultSet listTables(
3944
IDatabricksSession session,
4045
String catalog,
@@ -44,6 +49,7 @@ DatabricksResultSet listTables(
4449
throws SQLException;
4550

4651
/** Returns list of table types */
52+
@DatabricksMetricsTimed
4753
DatabricksResultSet listTableTypes(IDatabricksSession session) throws SQLException;
4854

4955
/**
@@ -59,6 +65,7 @@ DatabricksResultSet listTables(
5965
* absolute name)
6066
* @return a DatabricksResultSet representing list of columns
6167
*/
68+
@DatabricksMetricsTimed
6269
DatabricksResultSet listColumns(
6370
IDatabricksSession session,
6471
String catalog,
@@ -78,6 +85,7 @@ DatabricksResultSet listColumns(
7885
* absolute name)
7986
* @return a DatabricksResultSet representing list of functions
8087
*/
88+
@DatabricksMetricsTimed
8189
DatabricksResultSet listFunctions(
8290
IDatabricksSession session,
8391
String catalog,
@@ -94,6 +102,7 @@ DatabricksResultSet listFunctions(
94102
* @param table must match to a table in database
95103
* @return a DatabricksResultSet representing list of functions
96104
*/
105+
@DatabricksMetricsTimed
97106
DatabricksResultSet listPrimaryKeys(
98107
IDatabricksSession session, String catalog, String schema, String table) throws SQLException;
99108

@@ -106,6 +115,7 @@ DatabricksResultSet listPrimaryKeys(
106115
* @param table must match to a table in database
107116
* @return a DatabricksResultSet representing list of imported keys
108117
*/
118+
@DatabricksMetricsTimed
109119
DatabricksResultSet listImportedKeys(
110120
IDatabricksSession session, String catalog, String schema, String table) throws SQLException;
111121

@@ -118,6 +128,7 @@ DatabricksResultSet listImportedKeys(
118128
* @param table must match to a table in database
119129
* @return a DatabricksResultSet representing list of imported keys
120130
*/
131+
@DatabricksMetricsTimed
121132
DatabricksResultSet listExportedKeys(
122133
IDatabricksSession session, String catalog, String schema, String table) throws SQLException;
123134

@@ -133,6 +144,7 @@ DatabricksResultSet listExportedKeys(
133144
* @param foreignTable must match to a foreign table in database
134145
* @return a DatabricksResultSet representing list of cross references
135146
*/
147+
@DatabricksMetricsTimed
136148
DatabricksResultSet listCrossReferences(
137149
IDatabricksSession session,
138150
String parentCatalog,

0 commit comments

Comments
 (0)