From 2d36feb2b6a07e71ab947846f54bd0d63857c360 Mon Sep 17 00:00:00 2001
From: Heejong Lee <heejong@gmail.com>
Date: Thu, 12 May 2022 18:02:09 -0700
Subject: [PATCH] add comment

---
 .../apache/beam/model/pipeline/v1/beam_runner_api.proto   | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/model/pipeline/src/main/proto/org/apache/beam/model/pipeline/v1/beam_runner_api.proto b/model/pipeline/src/main/proto/org/apache/beam/model/pipeline/v1/beam_runner_api.proto
index 54d328cce82c..a19229a7857f 100644
--- a/model/pipeline/src/main/proto/org/apache/beam/model/pipeline/v1/beam_runner_api.proto
+++ b/model/pipeline/src/main/proto/org/apache/beam/model/pipeline/v1/beam_runner_api.proto
@@ -1046,12 +1046,8 @@ message StandardCoders {
     // Nullable types in container types (ArrayType, MapType) per the
     // encoding described for general Nullable types below.
     //
-    // Well known logical types:
-    //   beam:logical_type:micros_instant:v1
-    //   - Representation type: ROW<seconds: INT64, micros: INT64>
-    //   - A timestamp without a timezone where seconds + micros represents the
-    //     amount of time since the epoch.
-    //
+    // Logical types understood by all SDKs should be defined in schema.proto.
+    // Example of well known logical types:
     //   beam:logical_type:schema:v1
     //   - Representation type: BYTES
     //   - A Beam Schema stored as a serialized proto.