Copilot commented on code in PR #8958:
URL: https://github.com/apache/seatunnel/pull/8958#discussion_r2020423328


##########
seatunnel-connectors-v2/connector-cdc/connector-cdc-oracle/src/main/java/org/apache/seatunnel/connectors/seatunnel/cdc/oracle/utils/OracleConnectionUtils.java:
##########
@@ -120,4 +124,28 @@ public static List<TableId> listTables(
 
         return capturedTableIds;
     }
+
+    /** Show current container name in Oracle Server. */
+    public static String getCurrentContainerName(JdbcConnection 
jdbcConnection) {
+        try {
+            return jdbcConnection.queryAndMap(
+                    SHOW_CON_NAME,
+                    rs -> {
+                        if (rs.next()) {
+                            return rs.getString(1);
+                        } else {
+                            throw new SeaTunnelException(
+                                    "Cannot read the container name via '"
+                                            + SHOW_CON_NAME
+                                            + "'. Makesure your server is 
correctly configured");

Review Comment:
   Typo in error message: 'Makesure' should be 'Make sure'.
   ```suggestion
                                               + "'. Make sure your server is 
correctly configured");
   ```



##########
seatunnel-connectors-v2/connector-cdc/connector-cdc-oracle/src/main/java/org/apache/seatunnel/connectors/seatunnel/cdc/oracle/utils/OracleConnectionUtils.java:
##########
@@ -120,4 +124,28 @@ public static List<TableId> listTables(
 
         return capturedTableIds;
     }
+
+    /** Show current container name in Oracle Server. */
+    public static String getCurrentContainerName(JdbcConnection 
jdbcConnection) {
+        try {
+            return jdbcConnection.queryAndMap(
+                    SHOW_CON_NAME,
+                    rs -> {
+                        if (rs.next()) {
+                            return rs.getString(1);
+                        } else {
+                            throw new SeaTunnelException(
+                                    "Cannot read the container name via '"
+                                            + SHOW_CON_NAME
+                                            + "'. Makesure your server is 
correctly configured");
+                        }
+                    });
+        } catch (SQLException e) {
+            throw new SeaTunnelException(
+                    "Cannot read the container name via '"
+                            + SHOW_CON_NAME
+                            + "'. Makesure your server is correctly 
configured",

Review Comment:
   Typo in error message: 'Makesure' should be 'Make sure'.
   ```suggestion
                               + "'. Make sure your server is correctly 
configured",
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@seatunnel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to