docs: add samples for PostgresSQL #1781

Sivakumar-Searce · 2022-03-25T05:01:40Z

Added samples for PG dialect databases.

snippet-bot · 2022-03-25T05:01:43Z

Here is the summary of changes.

You are about to add 54 region tags.

samples/snippets/src/main/java/com/example/spanner/PgAsyncQueryToListAsyncExample.java:19, tag spanner_postgresql_async_query_to_list
samples/snippets/src/main/java/com/example/spanner/PgAsyncRunnerExample.java:19, tag spanner_postgresql_async_read_write_transaction
samples/snippets/src/main/java/com/example/spanner/PgAsyncTransactionManagerExample.java:19, tag spanner_postgresql_async_transaction_manager
samples/snippets/src/main/java/com/example/spanner/PgQueryWithNumericParameterSample.java:19, tag spanner_postgresql_query_with_numeric_parameter
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:100, tag spanner_postgresql_insert_data_with_timestamp_column
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:106, tag spanner_postgresql_insert_datatypes_data
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:195, tag spanner_postgresql_create_database
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:215, tag spanner_postgresql_insert_data
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:244, tag spanner_postgresql_delete_data
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:269, tag spanner_postgresql_query_data
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:284, tag spanner_postgresql_read_data
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:302, tag spanner_postgresql_add_column
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:326, tag spanner_postgresql_update_data
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:353, tag spanner_postgresql_read_write_transaction
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:398, tag spanner_postgresql_query_data_with_new_column
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:423, tag spanner_postgresql_create_index
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:446, tag spanner_postgresql_read_data_with_index
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:463, tag spanner_postgresql_create_storing_index
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:489, tag spanner_postgresql_read_data_with_storing_index
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:511, tag spanner_postgresql_read_only_transaction
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:539, tag spanner_postgresql_query_singers_table
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:558, tag spanner_postgresql_dml_getting_started_insert
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:577, tag spanner_postgresql_query_with_parameter
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:600, tag spanner_postgresql_dml_getting_started_update
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:654, tag spanner_postgresql_create_table_using_ddl
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:690, tag spanner_postgresql_read_stale_data
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:713, tag spanner_postgresql_update_data_with_timestamp_column
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:744, tag spanner_postgresql_add_timestamp_column
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:768, tag spanner_postgresql_query_data_with_timestamp_column
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:796, tag spanner_postgresql_create_table_with_timestamp_column
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:826, tag spanner_postgresql_insert_data_with_timestamp_column
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:871, tag spanner_postgresql_dml_standard_insert
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:886, tag spanner_postgresql_dml_standard_update
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:902, tag spanner_postgresql_dml_standard_delete
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:915, tag spanner_postgresql_dml_write_then_read
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:943, tag spanner_postgresql_dml_partitioned_update
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:951, tag spanner_postgresql_dml_partitioned_delete
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:959, tag spanner_postgresql_dml_batch_update
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:989, tag spanner_postgresql_create_table_with_datatypes
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1022, tag spanner_postgresql_insert_datatypes_data
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1050, tag spanner_postgresql_query_with_bool_parameter
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1073, tag spanner_postgresql_query_with_bytes_parameter
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1093, tag spanner_postgresql_query_with_float_parameter
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1116, tag spanner_postgresql_query_with_int_parameter
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1139, tag spanner_postgresql_query_with_string_parameter
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1158, tag spanner_postgresql_query_with_timestamp_parameter
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1180, tag spanner_postgresql_query_with_numeric_parameter
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1202, tag spanner_postgresql_create_client_with_query_options
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1229, tag spanner_postgresql_query_with_query_options
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1253, tag spanner_postgresql_list_backup_operations
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1286, tag spanner_postgresql_list_database_operations
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1518, tag spanner_init_client
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1535, tag spanner_init_client
samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java:1543, tag spanner_init_client

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

Refresh this comment

ansh0l · 2022-03-28T09:53:53Z

Approved presubmit checks to run.

Will wait for their results before proceeding with further review.

jin-jj · 2022-04-01T20:53:57Z

samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java

+    try (ResultSet resultSet =
+             dbClient
+                 .singleUse()
+                 .executeQuery(Statement.of("SELECT singerid as \"SingerId\", "


is the singlerid as \"\SingerId" used to keep the output looks same as before?

Or why not just SELECT SingerId, AlbumId, MarketingBudge FROM Albums?

jin-jj · 2022-04-01T20:53:57Z

samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java

+    try (ResultSet resultSet =
+             dbClient
+                 .singleUse()
+                 .executeQuery(Statement.of("SELECT singerid as \"SingerId\", "


is the singlerid as \"\SingerId" used to keep the output looks same as before?

Or why not just SELECT SingerId, AlbumId, MarketingBudge FROM Albums?

jin-jj · 2022-04-01T20:56:07Z

samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java

+    try (ResultSet resultSet =
+             dbClient
+                 .singleUse()
+                 .executeQuery(Statement.of("SELECT singerid as \"SingerId\", "


@ansh0l , I do not know much how the client handle the identifier, if we just use SELECT SingerId ...

would the resultSet.getLong("SingerId") work or not?

jin-jj · 2022-04-01T20:56:07Z

samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java

+    try (ResultSet resultSet =
+             dbClient
+                 .singleUse()
+                 .executeQuery(Statement.of("SELECT singerid as \"SingerId\", "


@ansh0l , I do not know much how the client handle the identifier, if we just use SELECT SingerId ...

would the resultSet.getLong("SingerId") work or not?

@jin-jj No, it would not work. If you execute SELECT SingerId FROM Singers, then PostgreSQL will fold the identifier to lower case, so it is effectively the same as executing SELECT singerid FROM Singers.

Identifiers that are quoted will be treated as case-sensitive and keep the actual lower/upper case letters.

jin-jj

Looks good. I think only question is how we wanna handle the case insensitive for PostgreSQL dialect. I saw we use singerid AS "SingerId" to workaround it.

Is this currently the best way to do it? Or should we just switch to lower case or just quote the Identifier always. @ansh0l Do you have an opinion for it?

jin-jj · 2022-04-06T14:19:50Z

samples/snippets/src/main/java/com/example/spanner/CustomTimeoutAndRetrySettingsExample.java

@@ -80,7 +80,7 @@ static void executeSqlWithCustomTimeoutAndRetrySettings(
          .readWriteTransaction()
          .run(transaction -> {
            String sql =
-                "INSERT Singers (SingerId, FirstName, LastName)\n"
+                "INSERT INTO Singers (SingerId, FirstName, LastName)\n"


Thanks fixing this syntax issue in old code.

jin-jj · 2022-04-06T14:22:18Z

samples/snippets/src/main/java/com/example/spanner/PgAsyncQueryToListAsyncExample.java

+
+package com.example.spanner;
+
+// [START spanner_pg_async_query_to_list]


Why only this tag has pg_ in between?
Our all tag naming may require update. I will update it here later what is our decision about the tag name for pg code samples.

For package name, we will use spanner_postgresql_*.

Pls fix this one and also apply the same pattern to all other tags.

jin-jj · 2022-04-06T14:27:32Z

samples/snippets/src/main/java/com/example/spanner/PgAsyncQueryToListAsyncExample.java

+    try (AsyncResultSet resultSet =
+             client
+                 .singleUse()
+                 .executeQueryAsync(Statement.of("SELECT singerid as \"SingerId\", "


The same as the comments, I am thinking renaming all column name in DDL from SingerId to singer_id, which is more pg friendly. I will circle back later to update what should we do.

We will keep the code as-is. No change required.

olavloite · 2022-04-06T13:36:35Z

samples/snippets/src/main/java/com/example/spanner/PgAsyncQueryToListAsyncExample.java

@@ -0,0 +1,90 @@
+/*
+ * Copyright 2020 Google Inc.


nit: 2022

(Please check everywhere else as well)

olavloite · 2022-04-06T15:19:49Z

samples/snippets/src/main/java/com/example/spanner/PgAsyncQueryToListAsyncExample.java

+ * limitations under the License.
+ */
+
+package com.example.spanner;


Would it make more sense to place all the PG samples in a separate sub-package. So something like com.example.spanner.postgresql? (I'm open to both solutions, both the one that is chosen now by prefixing the samples with Pg, as well as moving them to a separate package, but I would like us to take an informed decision on it)

Sure, will decide in today call with @jin-jj and decide the approach.

@olavloite : I think this was templated on https://github.com/googleapis/java-spanner/pull/1700/files.

IIRC, If we move to a sub-package - the class path would end up becoming something like samples/snippets/src/main/java/com/example/spanner/postgres/AsyncQueryToListAsyncExample.java, where the only gain probably will be readability of the class name, and some segregation from the regular GSQL samples.

The seggregation still happens in the current scheme of things, and looking for Spangres samples with an explicit pg in them sounds all right to me.

So we can keep things as is.

Thanks @olavloite and @ansh0l. So @Sivakumar-Searce we will keep the class name as-is.

olavloite · 2022-04-06T15:20:35Z

samples/snippets/src/main/java/com/example/spanner/PgAsyncQueryToListAsyncExample.java

+              executor);
+    }
+
+    for (AsyncQueryToListAsyncExample.Album album : albums.get(30L, TimeUnit.SECONDS)) {


Suggested change

for (AsyncQueryToListAsyncExample.Album album : albums.get(30L, TimeUnit.SECONDS)) {

for (Album album : albums.get(30L, TimeUnit.SECONDS)) {

olavloite · 2022-04-06T15:20:48Z

samples/snippets/src/main/java/com/example/spanner/PgAsyncQueryToListAsyncExample.java

+      albums =
+          resultSet.toListAsync(
+              reader -> {
+                return new AsyncQueryToListAsyncExample.Album(


Suggested change

return new AsyncQueryToListAsyncExample.Album(

return new Album(

olavloite · 2022-04-06T15:20:59Z

samples/snippets/src/main/java/com/example/spanner/PgAsyncQueryToListAsyncExample.java

+  static void asyncQueryToList(DatabaseClient client)
+      throws InterruptedException, ExecutionException, TimeoutException {
+    ExecutorService executor = Executors.newSingleThreadExecutor();
+    ApiFuture<? extends List<AsyncQueryToListAsyncExample.Album>> albums;


Suggested change

ApiFuture<? extends List<AsyncQueryToListAsyncExample.Album>> albums;

ApiFuture<? extends List<Album>> albums;

olavloite · 2022-04-06T15:34:20Z

samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java

+            "%d %s %s\n",
+            resultSet.getLong(0),
+            resultSet.getString(1),
+            // TODO: MarketingBudget uppercase not supported, keeping lowercase for now


Remove TODO before merging. Just using lower case here is OK with me (actually it's the most important part of the sample, as it shows an important difference between Spanner and PostgreSQL).

olavloite · 2022-04-06T15:36:35Z

samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java

+    try (ResultSet resultSet =
+             dbClient
+                 .singleUse()
+                 .executeQuery(Statement.of("SELECT singerid as \"SingerId\", "


@jin-jj No, it would not work. If you execute SELECT SingerId FROM Singers, then PostgreSQL will fold the identifier to lower case, so it is effectively the same as executing SELECT singerid FROM Singers.

Identifiers that are quoted will be treated as case-sensitive and keep the actual lower/upper case letters.

olavloite · 2022-04-06T15:38:26Z

samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java

+            dbId.getInstanceId().getInstance(),
+            dbId.getDatabase(),
+            Arrays.asList(
+                "ALTER TABLE Albums ADD COLUMN LastUpdateTime TIMESTAMPTZ"),


Are commit timestamps not (yet) supported?

I tried, it is not supported and couldn't find doc as well for commit timestamp for PG.

use spanner.commit_timestamp as the type when adding column

olavloite · 2022-04-06T15:44:05Z

samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java

+  }
+
+  // [START spanner_write_data_for_struct_queries]
+  static void writeStructExampleData(DatabaseClient dbClient) {


Maybe I'm missing something here, but I don't see any structs here. I also don't think we support structs for PostgreSQL databases, so this sample is probably void.

olavloite · 2022-04-06T15:46:35Z

samples/snippets/src/main/java/com/example/spanner/PgSpannerSample.java

+                "SELECT venueid as \"VenueId\", venuename as \"VenueName\" FROM Venues " + "WHERE"
+                    + " VenueInfo = $1")


nit:

Suggested change

"SELECT venueid as \"VenueId\", venuename as \"VenueName\" FROM Venues " + "WHERE"

+ " VenueInfo = $1")

"SELECT venueid as \"VenueId\", venuename as \"VenueName\" FROM Venues "

+ "WHERE VenueInfo = $1")

jin-jj

LGTM

olavloite

LGTM

jin-jj · 2022-04-15T15:23:43Z

Other than checkstyle fixes requested above, rest everything looks good. cc: @jin-jj

Thanks! @Sivakumar-Searce Please fix the issue as Anshul pointed out.

Sivakumar-Searce · 2022-04-19T03:46:14Z

@jin-jj @ansh0l check-style fixes suggested are fixed and committed the same. Please verify, Thanks !

ansh0l · 2022-04-19T03:48:09Z

@Sivakumar-Searce : Thanks! Approved presubmit checks to run.

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

… into spanner-pg-samples

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

ansh0l

LGTM

* chore: Adding release-please annotations to readme files Source-Link: googleapis/synthtool@327d46f Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:bf5639d265d70f6137d57d42ae781a6f4e26d4085ff4e018e71350480f9b3996

Added samples for PG dialect databases

376e449

Sivakumar-Searce requested a review from a team as a code owner March 25, 2022 05:01

product-auto-label bot added api: spanner Issues related to the googleapis/java-spanner API. samples Issues that are directly related to samples. labels Mar 25, 2022

blunderbuss-gcf bot assigned stephaniewang526 Mar 25, 2022

Sivakumar-Searce added 6 commits March 26, 2022 06:55

docs: add samples for postgresql

db81ea6

docs: samples tag fixes

5301349

docs: samples tag fixes

e764e9d

docs: samples tag fixes

332f15d

docs: samples tag violation fixes

45b1358

docs: samples imports and indent fixes

9d6b60e

docs: samples checkstyle fixes

2759e7a

jin-jj reviewed Apr 1, 2022

View reviewed changes

docs: add samples for postgresql

c4c2389

Sivakumar-Searce requested a review from a team as a code owner April 5, 2022 14:41

docs: samples tag fixes

daede6b

ansh0l requested a review from olavloite April 6, 2022 13:31

jin-jj reviewed Apr 6, 2022

View reviewed changes

olavloite reviewed Apr 6, 2022

View reviewed changes

docs: pg samples pr comments fixes

0a50828

jin-jj reviewed Apr 11, 2022

View reviewed changes

olavloite approved these changes Apr 14, 2022

View reviewed changes

ansh0l added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 14, 2022

docs: pg samples checkstyle fixes

1c900f1

product-auto-label bot added the size: xl Pull request size is extra large. label Apr 19, 2022

olavloite added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Apr 21, 2022

gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Apr 21, 2022

yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 21, 2022

gcf-owl-bot bot and others added 3 commits April 21, 2022 13:58

🦉 Updates from OwlBot post-processor

98a82e7

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Merge branch 'main' of https://github.com/Sivakumar-Searce/java-spanner…

04eaa97

… into spanner-pg-samples

docs: resolved readme merge conflicts

c8a9ccf

Sivakumar-Searce requested a review from ansh0l April 26, 2022 08:39

ansh0l added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2022

ansh0l unassigned stephaniewang526 Apr 29, 2022

yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2022

ansh0l added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2022

yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2022

ansh0l added kokoro:run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Apr 29, 2022

gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Apr 29, 2022

ansh0l removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Apr 29, 2022

🦉 Updates from OwlBot post-processor

64d87ab

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

ansh0l added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Apr 29, 2022

yoshi-kokoro removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Apr 29, 2022

ansh0l approved these changes Apr 29, 2022

View reviewed changes

ansh0l merged commit e832298 into googleapis:main Apr 29, 2022

release-please bot mentioned this pull request Apr 29, 2022

chore(main): release 6.24.0 #1846

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

docs: add samples for PostgresSQL #1781

docs: add samples for PostgresSQL #1781

Sivakumar-Searce commented Mar 25, 2022

snippet-bot bot commented Mar 25, 2022 •

edited

ansh0l commented Mar 28, 2022

jin-jj Apr 1, 2022

jin-jj Apr 1, 2022

jin-jj Apr 1, 2022

jin-jj Apr 1, 2022

olavloite Apr 6, 2022

jin-jj left a comment

jin-jj Apr 6, 2022

jin-jj Apr 6, 2022

jin-jj Apr 7, 2022

jin-jj Apr 6, 2022

jin-jj Apr 7, 2022

olavloite Apr 6, 2022

olavloite Apr 6, 2022

Sivakumar-Searce Apr 7, 2022

ansh0l Apr 7, 2022

jin-jj Apr 7, 2022

olavloite Apr 6, 2022

olavloite Apr 6, 2022

olavloite Apr 6, 2022

olavloite Apr 6, 2022

olavloite Apr 6, 2022

olavloite Apr 6, 2022

Sivakumar-Searce Apr 7, 2022

jin-jj Apr 7, 2022

olavloite Apr 6, 2022

olavloite Apr 6, 2022

jin-jj left a comment

olavloite left a comment

jin-jj commented Apr 15, 2022

Sivakumar-Searce commented Apr 19, 2022

ansh0l commented Apr 19, 2022

ansh0l left a comment


		package com.example.spanner;

		// [START spanner_pg_async_query_to_list]

	for (AsyncQueryToListAsyncExample.Album album : albums.get(30L, TimeUnit.SECONDS)) {
	for (Album album : albums.get(30L, TimeUnit.SECONDS)) {

	return new AsyncQueryToListAsyncExample.Album(
	return new Album(

	ApiFuture<? extends List<AsyncQueryToListAsyncExample.Album>> albums;
	ApiFuture<? extends List<Album>> albums;

		"SELECT venueid as \"VenueId\", venuename as \"VenueName\" FROM Venues " + "WHERE"
		+ " VenueInfo = $1")

docs: add samples for PostgresSQL #1781

docs: add samples for PostgresSQL #1781

Conversation

Sivakumar-Searce commented Mar 25, 2022

snippet-bot bot commented Mar 25, 2022 • edited

ansh0l commented Mar 28, 2022

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jin-jj left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jin-jj left a comment

Choose a reason for hiding this comment

olavloite left a comment

Choose a reason for hiding this comment

jin-jj commented Apr 15, 2022

Sivakumar-Searce commented Apr 19, 2022

ansh0l commented Apr 19, 2022

ansh0l left a comment

Choose a reason for hiding this comment

snippet-bot bot commented Mar 25, 2022 •

edited