maksaska commented on code in PR #317:
URL: https://github.com/apache/ignite-extensions/pull/317#discussion_r2273784655
##########
modules/cdc-ext/src/main/java/org/apache/ignite/cdc/postgresql/JavaToSqlTypeMapper.java:
##########
@@ -0,0 +1,221 @@
+package org.apache.ignite.cdc.postgresql;
+
+import java.math.BigDecimal;
+import java.sql.PreparedStatement;
+import java.sql.Types;
+import java.time.LocalDate;
+import java.time.LocalDateTime;
+import java.time.LocalTime;
+import java.time.OffsetDateTime;
+import java.time.OffsetTime;
+import java.util.HashMap;
+import java.util.Map;
+import java.util.UUID;
+import org.apache.ignite.IgniteException;
+
+import static
org.apache.ignite.cdc.postgresql.JavaToSqlTypeMapper.JavaToSqlType.OBJECT;
+
+/** */
+class JavaToSqlTypeMapper {
+ /** */
+ private static final Map<String, JavaToSqlType> JAVA_TO_SQL_TYPE_MAP = new
HashMap<>();
+
+ static {
+ for (JavaToSqlType type : JavaToSqlType.values())
+ JAVA_TO_SQL_TYPE_MAP.put(type.javaTypeName(), type);
+ }
+
+ /**
+ * Sets a value in the PreparedStatement at the given index using the
appropriate setter
+ * based on the runtime type of the object.
+ * @param stmt {@link PreparedStatement}
+ * @param idx value index in {@link PreparedStatement}
+ * @param obj value
+ */
+ public void setValue(PreparedStatement stmt, int idx, Object obj) {
+ try {
+ if (obj == null) {
+ stmt.setNull(idx, Types.NULL);
+
+ return;
+ }
+
+ if (obj instanceof byte[]) {
+ stmt.setBytes(idx, (byte[])obj); // Preferred setter for byte[]
+
+ return;
+ }
+
+ JavaToSqlType type =
JAVA_TO_SQL_TYPE_MAP.getOrDefault(obj.getClass().getName(), OBJECT);
+
+ if (type != null)
+ stmt.setObject(idx, obj, type.typeId());
+ else
+ stmt.setObject(idx, obj);
+ }
+ catch (Throwable e) {
+ throw new IgniteException("Failed to set value for type!", e);
+ }
+ }
+
+ /**
+ * Renders the SQL type declaration for a given Java class name based on
its mapping,
+ * optionally including precision and scale if the SQL type supports them.
+ *
+ * @param clsName the fully qualified Java class name used to look up the
corresponding SQL type
+ * @param precision optional precision value to include in the SQL type,
if supported
+ * @param scale optional scale value to include in the SQL type, if
supported
+ * @return the SQL type string with appropriate precision and scale
formatting
+ */
+ public String renderSqlType(String clsName, Integer precision, Integer
scale) {
+ JavaToSqlType type = JAVA_TO_SQL_TYPE_MAP.getOrDefault(clsName,
OBJECT);
+
+ if (type.precision()) {
+ if (type.scale()) {
+ if (precision != null && scale != null)
+ return type.sqlType().replace("?", String.format("(%d,
%d)", precision, scale));
Review Comment:
It's was needed for INTERVAL types. Since we removed it from this patch,
there is no need for such handling anymore. Fixed
--
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: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
