extend existing proposal with comment suggestion

Signed-off-by: Pawel Leszczynski <leszczynski.pawel@gmail.com>

proposals/2045-column-lineage-endpoint.md

@@ -1,6 +1,6 @@
 # Proposal: Column lineage endpoint proposal
 
-Author(s): @julienledem
+Author(s): @julienledem, @pawel-big-lebowski
 
 Created: 20022-08-18
 
@@ -18,16 +18,24 @@ Dicussion: [column lineage endpoint issue #2045](https://github.com/MarquezProje
 - OpenLineage defines a [column-level lineage facet]- (https://github.com/OpenLineage/OpenLineage/blob/ff0d87d30ed6c9fe39472788948266a6d3190585/spec/facets/ColumnLineageDatasetFacet.md). 
 - Marquez has a lineage endpoint `GET /api/v1/lineage` that returns the current lineage graph connected to a job or a dataset
 
+### Column lineage characteristics and general assumptions
+
+Column level lineage is a different lineage graph due to a different node granularity - kind of zoomed-in view of existing lineage. Instead of datasets and jobs being lineage graph nodes, each dataset field becomes a node. Additionally, there are edges between dataset fields, instead of datasets itself. Thus, enriching existing lineage with column lineage information would not be sufficient. That’s why we propose another API endpoint with column lineage graph.
+
+Upstream and downstream edges do have different characteristics. An output dataset is always produced by a single version of input dataset (one upstream), while a single input datset version can have multiple output dataset versions. Lineage graph can be then easily flooded by downstream subgraph which blurs the overall view. That's why we consider an upstream column lineage as a default one. Downstream lineage will be returned only when requested explicitly. 
+
 ### New Elements
+
 We propose to add the following:
-- Add column lineage to the lineage endpoint
-- A new column-lineage endpoint leveraging the column lineage facet to retrieve lineage for a given column.
-- Point-in-time upstream (dataset or column level) lineage given a version of a dataset.
+
+ - Add column lineage to the lineage endpoint as a part of dataset.
+ - A new column-lineage endpoint leveraging the column lineage facet to retrieve lineage for a given column.
+ - Point-in-time upstream (dataset or column level) lineage given a version of a dataset.
 
 ## Proposal
 
-### add column lineage to existing endpoint
-In the GET /lineage api, add column lineage to DATASET nodes' data
+### Add column lineage to existing endpoint
+In the `GET /lineage` api, add column lineage to `DATASET` nodes' data:
 ```diff
 {
   "id": "dataset:food_delivery:public.categories",
@@ -66,78 +74,116 @@ In the GET /lineage api, add column lineage to DATASET nodes' data
 }
 ```
 
-### add a column-level-lineage endpoint:
+The implementation here can reuse `columnLineage` facet classes.
+
+
+### Add a column-level-lineage endpoint:
 
+New endpoints to retrieve a column lineage of a single field or a whole dataset will be added: 
 ```
-GET /column-lineage?nodeId=dataset:food_delivery:public.delivery_7_days&column=a
+GET /column-lineage?nodeId=dataset:{namespace}:{dataset}
+GET /column-lineage?nodeId=datasetField:{namespace}:{dataset}:{field}
+```
+For example:
+```
+GET /column-lineage?nodeId=dataset:food_delivery:public.delivery_7_days
+GET /column-lineage?nodeId=datasetField:food_delivery:public.delivery_7_days:a
 ```
-`column` is a ne parameter that must be a column in the schema of the provided dataset `nodeId`.
 
-The logic is layered on the existing lineage endpoint, filtering down to the datasets that contribute to that column.
-It only returns dataset nodes.
+Although creating a new endpoint, we would like to reuse existing data structures with a new `NodeType.FIELD` introduced.
 
-```diff
+The logic returns dataset field node:
+
+```
+GET /column-lineage?nodeId=datasetField:db1:table1:a
+...
 {
   graph: [
     {
-       "id": "dataset:db1:table2",
-       "type": "DATASET",
-       data: {
-         namespace: "DB1",
-         name: "table2",
->       columnLineage: {
->           "a": {
->              inputFields: [
->                {namespace: "DB1", name: "table1, "field": "a"}
->              ],
->              transformationDescription: "identical",
->              transformationType: "IDENTITY"
->           },
->          "b": ... other output fields
->        }
+       "id": "datasetField:db1:table1:a",
+       "type": "DATASET_FIELD",
+       "data": {
+         "namespace": "DB1",
+         "name": "table2",
+         "field": "a",
+         "type": "integer",
+         "transformationDescription": "identical",
+         "transformationType": "IDENTITY",    
+         "inputFields": [
+            { "namespace": "DBA", "name": "tableA", "field": "columnA"},
+            { "namespace": "DBB", "name": "tableB", "field": "columnB"},
+            { "namespace": "DBC", "name": "tableC", "field": "columnC"}
+         ]
+         "inEdges": [
+          {
+             "origin": "datasetField:db1:table1:a",
+             "destination": "datasetField:DBA:tableA:columnA"
+          },
+          {
+             "origin": "datasetField:db1:table1:a",
+             "destination": "datasetField:DBB:tableB:columnB"
+          },
+          {
+             "origin": "datasetField:db1:table1:a",
+             "destination": "datasetField:DBB:tableB:columnC"
+          }
+         ],
       },
       ...
+      # Input fields, present within "inEdges", can be also returned within a graph due to a `depth` parameter greate than 0.
   }
   ]
 }  
 ```
 
+The `depth` parameter controls how many edges, from a given dataset field, shall be returned. The default is set to `0`. In case of default equal `1`, each `inputField` will be returned as a separate node within a response graph with `inputFields` used to produce it. Please note that extending depth may increase the graph size and affect request performance. 
+
+The endpoints above fetches upstream column-lineage for given dataset field or all fields within a dataset. Downstream column lineage is turned off by default. However, this can be turned on with an extra `withDownstream` parameter like: 
+
+```
+GET /column-lineage?nodeId=datasetField:food_delivery:public.delivery_7_days:a&withDownstream=true
+
+```
+This will include `outEdges` within the returned node of the graph.
+
+
 ### Point in time upstream lineage
-return historical upstream lineage from a given Dataset version. 
-This adds the version element to the nodeId in both the existing `/api/v1/lineage` and newly proposed `/api/v1/column-lineage` endpoint 
+
+Changes related to `/api/v1/lineage` are out of scope of this proposal which will only include point in time lineage for newly proposed `/api/v1/column-lineage` endpoint:
 ```
-GET /lineage?nodeId=dataset:food_delivery:public.delivery_7_days:{version}
-GET /column-lineage?nodeId=dataset:food_delivery:public.delivery_7_days:{version}&column=a
+GET /column-lineage?nodeId=dataset_field:food_delivery:public.delivery_7_days:a&datasetVersion=123e4567-e89b-12d3-a456-426614174000
+GET /column-lineage?nodeId=dataset_field:food_delivery:public.delivery_7_days:a&lineageAt=1661846242
 ```
-This returns only upstream lineage in this current proposal. This is because upstream lineage is well defined to a specific version while downstream lineage is not. The data payload would add a version field.
+
+Point in time can be controlled by:
+ * **datasetVersion** - uuid of a specific dataset version,
+ * **lineageAt** - which contains a unix timestamp.
+
+When **lineageAt** specified, the latest dataset version before timestamp will be found. Regardles **datasetVersion** or **lineageAt** parameters applied, responses will be the same as below: 
+
 ```diff
 {
   graph: [
     {
-<      "id": "dataset:db1:table2",
->      "id": "dataset:db1:table2#{VERSION UUID}",
-       "type": "DATASET",
-       data: {
-         namespace: "DB1",
-         name: "table2",
->        version: "{VERSION UUID}"
-         ...
-       }
-    }
-  ]
+<       "id": "datasetField:db1:table1:a",
+>       "id": "datasetField:db1:table1:a#{VERSION UUID}",
+       "type": "DATASET_FIELD",
+       "data": {
+        ....
 }  
 ```
 
 ## Implementation
 
 ### columne lineage facet in lineage
-Adding the columnLineage facet requires a formatting of existing facet data. 
+Adding the columnLineage facet requires a formatting of existing facet data (work in progress).
 ### column lineage endpoint
 The `/column-lineage` endpoint leverages the `/lineage` endpoint and then filters down the payload to return the expected result.
 ### point-in-time upstream lineage
+
 The point-in-time upstream lineage leverages the run to dataset version relation to track back the lineage of a given dataset of job version.
 Dataset version -> run that produced it -> consumed Dataset Versions.
 
 ## Next Steps
 
-Review of this proposal and production of detailed design for the implementation, in particular for the point in time lineage which might affect the dabtabase schema.
+Review of this proposal and production of detailed design for the implementation.: