Skip to content

Commit

Permalink
Remove unused gRPC method for Pandas DataFrames (#3156)
Browse files Browse the repository at this point in the history
This has been superceeded by FetchObject and a plugin for Pandas.

Fixes #3153
  • Loading branch information
niloc132 committed Jan 17, 2023
1 parent 7926faf commit 5da4064
Show file tree
Hide file tree
Showing 120 changed files with 2,878 additions and 3,380 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/**
* Copyright (c) 2016 - 2022 Deephaven Data Labs and Patent Pending
* Copyright (c) 2016-2022 Deephaven Data Labs and Patent Pending
*/
package io.deephaven.auth;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
import io.deephaven.proto.backplane.grpc.EmptyTableRequest;
import io.deephaven.proto.backplane.grpc.ExactJoinTablesRequest;
import io.deephaven.proto.backplane.grpc.ExportedTableUpdatesRequest;
import io.deephaven.proto.backplane.grpc.FetchPandasTableRequest;
import io.deephaven.proto.backplane.grpc.FetchTableRequest;
import io.deephaven.proto.backplane.grpc.FilterTableRequest;
import io.deephaven.proto.backplane.grpc.FlattenRequest;
Expand Down Expand Up @@ -70,17 +69,6 @@ void checkPermissionGetExportedTableCreationResponse(AuthContext authContext, Ti
void checkPermissionFetchTable(AuthContext authContext, FetchTableRequest request,
List<Table> sourceTables);

/**
* Authorize a request to FetchPandasTable.
*
* @param authContext the authentication context of the request
* @param request the request to authorize
* @param sourceTables the operation's source tables
* @throws io.grpc.StatusRuntimeException if the user is not authorized to invoke FetchPandasTable
*/
void checkPermissionFetchPandasTable(AuthContext authContext, FetchPandasTableRequest request,
List<Table> sourceTables);

/**
* Authorize a request to ApplyPreviewColumns.
*
Expand Down Expand Up @@ -473,9 +461,6 @@ public void checkPermissionGetExportedTableCreationResponse(AuthContext authCont
public void checkPermissionFetchTable(AuthContext authContext, FetchTableRequest request,
List<Table> sourceTables) {}

public void checkPermissionFetchPandasTable(AuthContext authContext,
FetchPandasTableRequest request, List<Table> sourceTables) {}

public void checkPermissionApplyPreviewColumns(AuthContext authContext,
ApplyPreviewColumnsRequest request, List<Table> sourceTables) {}

Expand Down Expand Up @@ -593,11 +578,6 @@ public void checkPermissionFetchTable(AuthContext authContext, FetchTableRequest
ServiceAuthWiring.operationNotAllowed();
}

public void checkPermissionFetchPandasTable(AuthContext authContext,
FetchPandasTableRequest request, List<Table> sourceTables) {
ServiceAuthWiring.operationNotAllowed();
}

public void checkPermissionApplyPreviewColumns(AuthContext authContext,
ApplyPreviewColumnsRequest request, List<Table> sourceTables) {
ServiceAuthWiring.operationNotAllowed();
Expand Down Expand Up @@ -791,13 +771,6 @@ public void checkPermissionFetchTable(AuthContext authContext, FetchTableRequest
}
}

public void checkPermissionFetchPandasTable(AuthContext authContext,
FetchPandasTableRequest request, List<Table> sourceTables) {
if (delegate != null) {
delegate.checkPermissionFetchPandasTable(authContext, request, sourceTables);
}
}

public void checkPermissionApplyPreviewColumns(AuthContext authContext,
ApplyPreviewColumnsRequest request, List<Table> sourceTables) {
if (delegate != null) {
Expand Down
Loading

0 comments on commit 5da4064

Please sign in to comment.