feat(executors): add Fetch endpoint to gRPC server

Author: AndreyMarkinPPC

libs/executors/garf/executors/entrypoints/grpc_server.py

@@ -41,6 +41,22 @@ def Execute(self, request, context):
     )
     return garf_pb2.ExecuteResponse(results=[result])
 
+  def Fetch(self, request, context):
+    query_executor = garf.executors.setup_executor(
+      request.source, request.context.fetcher_parameters
+    )
+    execution_context = garf.executors.execution_context.ExecutionContext(
+      **MessageToDict(request.context, preserving_proto_field_name=True)
+    )
+    result = query_executor.fetcher.fetch(
+      query_specification=request.query,
+      title=request.title,
+      args=execution_context.query_parameters,
+    )
+    return garf_pb2.FetchResponse(
+      columns=result.column_names, rows=result.to_list(row_type='dict')
+    )
+
 
 if __name__ == '__main__':
   parser = argparse.ArgumentParser()

libs/executors/garf/executors/garf_pb2.py

@@ -5,7 +5,7 @@
 
 from . import garf_pb2 as garf__pb2
 
-GRPC_GENERATED_VERSION = '1.75.0'
+GRPC_GENERATED_VERSION = '1.76.0'
 GRPC_VERSION = grpc.__version__
 _version_not_supported = False
 
@@ -18,7 +18,7 @@
 if _version_not_supported:
     raise RuntimeError(
         f'The grpc package installed is at version {GRPC_VERSION},'
-        + f' but the generated code in garf_pb2_grpc.py depends on'
+        + ' but the generated code in garf_pb2_grpc.py depends on'
         + f' grpcio>={GRPC_GENERATED_VERSION}.'
         + f' Please upgrade your grpc module to grpcio>={GRPC_GENERATED_VERSION}'
         + f' or downgrade your generated code using grpcio-tools<={GRPC_VERSION}.'
@@ -39,6 +39,11 @@ def __init__(self, channel):
                 request_serializer=garf__pb2.ExecuteRequest.SerializeToString,
                 response_deserializer=garf__pb2.ExecuteResponse.FromString,
                 _registered_method=True)
+        self.Fetch = channel.unary_unary(
+                '/garf.GarfService/Fetch',
+                request_serializer=garf__pb2.FetchRequest.SerializeToString,
+                response_deserializer=garf__pb2.FetchResponse.FromString,
+                _registered_method=True)
 
 
 class GarfServiceServicer(object):
@@ -50,6 +55,12 @@ def Execute(self, request, context):
         context.set_details('Method not implemented!')
         raise NotImplementedError('Method not implemented!')
 
+    def Fetch(self, request, context):
+        """Missing associated documentation comment in .proto file."""
+        context.set_code(grpc.StatusCode.UNIMPLEMENTED)
+        context.set_details('Method not implemented!')
+        raise NotImplementedError('Method not implemented!')
+
 
 def add_GarfServiceServicer_to_server(servicer, server):
     rpc_method_handlers = {
@@ -58,6 +69,11 @@ def add_GarfServiceServicer_to_server(servicer, server):
                     request_deserializer=garf__pb2.ExecuteRequest.FromString,
                     response_serializer=garf__pb2.ExecuteResponse.SerializeToString,
             ),
+            'Fetch': grpc.unary_unary_rpc_method_handler(
+                    servicer.Fetch,
+                    request_deserializer=garf__pb2.FetchRequest.FromString,
+                    response_serializer=garf__pb2.FetchResponse.SerializeToString,
+            ),
     }
     generic_handler = grpc.method_handlers_generic_handler(
             'garf.GarfService', rpc_method_handlers)
@@ -95,3 +111,30 @@ def Execute(request,
             timeout,
             metadata,
             _registered_method=True)
+
+    @staticmethod
+    def Fetch(request,
+            target,
+            options=(),
+            channel_credentials=None,
+            call_credentials=None,
+            insecure=False,
+            compression=None,
+            wait_for_ready=None,
+            timeout=None,
+            metadata=None):
+        return grpc.experimental.unary_unary(
+            request,
+            target,
+            '/garf.GarfService/Fetch',
+            garf__pb2.FetchRequest.SerializeToString,
+            garf__pb2.FetchResponse.FromString,
+            options,
+            channel_credentials,
+            insecure,
+            call_credentials,
+            compression,
+            wait_for_ready,
+            timeout,
+            metadata,
+            _registered_method=True)

libs/executors/garf/executors/garf_pb2_grpc.py

@@ -1,3 +1,3 @@
 python -m grpc_tools.protoc -I=../../protos/ \
-	--python_out=./garf_executors --grpc_python_out=./garf_executors \
+	--python_out=./garf/executors --grpc_python_out=./garf/executors \
 	../../protos/garf.proto

libs/executors/generate_protos.sh

@@ -57,6 +57,7 @@ server=[
   "fastapi[standard]",
   "opentelemetry-instrumentation-fastapi",
   "typer",
+  "grpcio-reflection",
 ]
 all = [
   "garf-executors[bq,sql,server,gcp]"

libs/executors/pyproject.toml

@@ -11,24 +11,80 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
+import pathlib
+
 import grpc
 import pytest
 from garf.executors import garf_pb2 as pb
 from garf.executors import garf_pb2_grpc
+from google.protobuf.json_format import MessageToDict
+
+_SCRIPT_PATH = pathlib.Path(__file__).parent
+_QUERY = """
+  SELECT
+    resource,
+    dimensions.name AS name,
+    metrics.clicks AS clicks
+  FROM resource
+"""
+expected_output = [
+  {
+    'resource': 'Campaign A',
+    'name': 'Ad Group 1',
+    'clicks': 1500,
+  },
+  {
+    'resource': 'Campaign B',
+    'name': 'Ad Group 2',
+    'clicks': 2300,
+  },
+  {
+    'resource': 'Campaign C',
+    'name': 'Ad Group 3',
+    'clicks': 800,
+  },
+  {
+    'resource': 'Campaign A',
+    'name': 'Ad Group 4',
+    'clicks': 3200,
+  },
+]
 
 
-@pytest.mark.skip
-def test_grpc_call():
+def test_grpc_execute():
+  fake_data = _SCRIPT_PATH / 'test.json'
   with grpc.insecure_channel('localhost:50051') as channel:
     stub = garf_pb2_grpc.GarfServiceStub(channel)
     request = pb.ExecuteRequest(
-      source='rest',
+      source='fake',
       title='example',
-      query='SELECT id, name AS model, data.color AS color FROM objects',
+      query=_QUERY,
       context=pb.ExecutionContext(
-        fetcher_parameters={'endpoint': 'https://api.restful-api.dev'},
+        fetcher_parameters={
+          'data_location': str(fake_data),
+        },
         writer='csv',
       ),
     )
     result = stub.Execute(request)
     assert 'CSV' in result.results[0]
+
+
+def test_grpc_fetch(tmp_path):
+  fake_data = _SCRIPT_PATH / 'test.json'
+  with grpc.insecure_channel('localhost:50051') as channel:
+    stub = garf_pb2_grpc.GarfServiceStub(channel)
+    request = pb.FetchRequest(
+      source='fake',
+      title='example',
+      query=_QUERY,
+      context=pb.FetchContext(
+        fetcher_parameters={
+          'data_location': str(fake_data),
+        }
+      ),
+    )
+    result = stub.Fetch(request)
+    data = [MessageToDict(r) for r in result.rows]
+    assert result.columns == ['resource', 'name', 'clicks']
+    assert data == expected_output

libs/executors/tests/end-to-end/test_grpc_server.py

@@ -5,6 +5,25 @@ import "google/protobuf/struct.proto";
 
 service GarfService {
   rpc Execute(ExecuteRequest) returns (ExecuteResponse) {}
+  rpc Fetch(FetchRequest) returns (FetchResponse) {}
+
+}
+
+message FetchRequest {
+  string source = 1;
+  string title = 2;
+  string query = 3;
+  FetchContext context = 4;
+}
+
+message FetchResponse {
+  repeated string columns = 1;
+  repeated google.protobuf.Struct rows = 2;
+}
+
+message FetchContext {
+  QueryParameters query_parameters = 1;
+  google.protobuf.Struct fetcher_parameters = 2;
 }
 
 message ExecuteRequest {