From 7a3a9877afbab9d6645b954d8e750fe303664e8e Mon Sep 17 00:00:00 2001
From: zero88 <sontt246@gmail.com>
Date: Sun, 10 Apr 2022 20:01:47 +0700
Subject: [PATCH] [#101] Add @Deprecated for LegacyJooqx

---
 .../java/io/github/zero88/jooqx/LegacyJooqx.java |  5 +++++
 .../github/zero88/jooqx/LegacyJooqxBuilder.java  |  1 +
 .../io/github/zero88/jooqx/LegacyJooqxTx.java    |  1 +
 .../github/zero88/jooqx/LegacySQLCollector.java  |  1 +
 .../io/github/zero88/jooqx/LegacySQLImpl.java    | 16 +++++++++++++---
 .../zero88/jooqx/LegacySQLPreparedQuery.java     |  1 +
 6 files changed, 22 insertions(+), 3 deletions(-)

diff --git a/core/src/main/java/io/github/zero88/jooqx/LegacyJooqx.java b/core/src/main/java/io/github/zero88/jooqx/LegacyJooqx.java
index c577a16f..b496de7f 100644
--- a/core/src/main/java/io/github/zero88/jooqx/LegacyJooqx.java
+++ b/core/src/main/java/io/github/zero88/jooqx/LegacyJooqx.java
@@ -24,8 +24,13 @@
  * @see JDBCClient
  * @see ResultSet
  * @since 1.0.0
+ * @deprecated Since <a href="https://vertx.io/docs/vertx-jdbc-client/java/#_legacy_jdbc_client_api">Vertx
+ *     deprecated</a>, this interface still works but not able to using with
+ *     <a href="https://vertx.io/docs/vertx-rx/java3/">Rxify version 3</a> and
+ *     <a href="https://smallrye.io/smallrye-mutiny-vertx-bindings/2.20.0/">Vertx mutiny</a>
  */
 @VertxGen
+@Deprecated
 public interface LegacyJooqx extends LegacyInternal<SQLClient> {
 
     /**
diff --git a/core/src/main/java/io/github/zero88/jooqx/LegacyJooqxBuilder.java b/core/src/main/java/io/github/zero88/jooqx/LegacyJooqxBuilder.java
index 69492fe1..2f39a77f 100644
--- a/core/src/main/java/io/github/zero88/jooqx/LegacyJooqxBuilder.java
+++ b/core/src/main/java/io/github/zero88/jooqx/LegacyJooqxBuilder.java
@@ -18,6 +18,7 @@
  * @since 2.0.0
  */
 @VertxGen
+@Deprecated
 public interface LegacyJooqxBuilder extends
                                     SQLExecutorBuilder<SQLClient, JsonArray, LegacySQLPreparedQuery, ResultSet,
                                                           LegacySQLCollector, LegacyJooqxBuilder> {
diff --git a/core/src/main/java/io/github/zero88/jooqx/LegacyJooqxTx.java b/core/src/main/java/io/github/zero88/jooqx/LegacyJooqxTx.java
index b2f68b0c..bac0a5b0 100644
--- a/core/src/main/java/io/github/zero88/jooqx/LegacyJooqxTx.java
+++ b/core/src/main/java/io/github/zero88/jooqx/LegacyJooqxTx.java
@@ -26,6 +26,7 @@
  * @since 1.0.0
  */
 @VertxGen
+@Deprecated
 public interface LegacyJooqxTx extends LegacyInternal<SQLConnection>,
                                        SQLTxExecutor<SQLConnection, JsonArray, LegacySQLPreparedQuery, ResultSet,
                                                         LegacySQLCollector, LegacyJooqxTx> {
diff --git a/core/src/main/java/io/github/zero88/jooqx/LegacySQLCollector.java b/core/src/main/java/io/github/zero88/jooqx/LegacySQLCollector.java
index b135cad8..f6d76337 100644
--- a/core/src/main/java/io/github/zero88/jooqx/LegacySQLCollector.java
+++ b/core/src/main/java/io/github/zero88/jooqx/LegacySQLCollector.java
@@ -16,6 +16,7 @@
  * @since 1.0.0
  */
 @VertxGen
+@Deprecated
 public interface LegacySQLCollector extends SQLResultCollector<ResultSet>, SQLBatchCollector<ResultSet, List<Integer>> {
 
     static LegacySQLCollector create() {
diff --git a/core/src/main/java/io/github/zero88/jooqx/LegacySQLImpl.java b/core/src/main/java/io/github/zero88/jooqx/LegacySQLImpl.java
index 88959067..cff05f29 100644
--- a/core/src/main/java/io/github/zero88/jooqx/LegacySQLImpl.java
+++ b/core/src/main/java/io/github/zero88/jooqx/LegacySQLImpl.java
@@ -28,6 +28,7 @@
 import io.github.zero88.jooqx.adapter.SQLResultAdapter;
 import io.github.zero88.jooqx.adapter.SelectStrategy;
 import io.github.zero88.jooqx.datatype.DataTypeMapperRegistry;
+import io.vertx.codegen.annotations.Nullable;
 import io.vertx.core.Future;
 import io.vertx.core.Promise;
 import io.vertx.core.Vertx;
@@ -38,8 +39,10 @@
 import io.vertx.ext.sql.SQLOperations;
 import io.vertx.ext.sql.UpdateResult;
 
+@Deprecated
 final class LegacySQLImpl {
 
+    @Deprecated
     interface LegacyInternal<S extends SQLOperations>
         extends SQLExecutor<S, JsonArray, LegacySQLPreparedQuery, ResultSet, LegacySQLCollector> {
 
@@ -52,6 +55,7 @@ interface LegacyInternal<S extends SQLOperations>
     }
 
 
+    @Deprecated
     static final class LegacySQLPQ extends SQLPQ<JsonArray> implements LegacySQLPreparedQuery {
 
         @Override
@@ -68,6 +72,7 @@ protected JsonArray doConvert(Map<String, Param<?>> params, DataTypeMapperRegist
     }
 
 
+    @Deprecated
     static final class LegacySQLRC implements LegacySQLCollector {
 
         @NotNull
@@ -106,6 +111,7 @@ private Map<Field<?>, Integer> getColumnMap(ResultSet rs, Function<String, Field
     }
 
 
+    @Deprecated
     abstract static class LegacySQLEI<S extends SQLOperations>
         extends SQLEI<S, JsonArray, LegacySQLPreparedQuery, ResultSet, LegacySQLCollector>
         implements LegacyInternal<S> {
@@ -117,7 +123,7 @@ abstract static class LegacySQLEI<S extends SQLOperations>
         }
 
         @Override
-        public final <T, R> Future<R> execute(@NotNull Query query, @NotNull SQLResultAdapter<T, R> adapter) {
+        public final <T, R> Future<@Nullable R> execute(@NotNull Query query, @NotNull SQLResultAdapter<T, R> adapter) {
             final Promise<ResultSet> promise = Promise.promise();
             sqlClient().queryWithParams(preparedQuery().sql(dsl().configuration(), query),
                                         preparedQuery().bindValues(query, typeMapperRegistry()), promise);
@@ -158,6 +164,7 @@ public Future<Integer> ddl(@NotNull DDLQuery statement) {
     }
 
 
+    @Deprecated
     static final class LegacyJooqxImpl extends LegacySQLEI<SQLClient> implements LegacyJooqx {
 
         LegacyJooqxImpl(Vertx vertx, DSLContext dsl, SQLClient sqlClient, LegacySQLPreparedQuery preparedQuery,
@@ -190,6 +197,7 @@ protected Future<SQLConnection> openConn() {
     }
 
 
+    @Deprecated
     static final class LegacyJooqTxImpl extends LegacySQLEI<SQLConnection> implements LegacyJooqxTx {
 
         private final LegacySQLEI<SQLClient> delegate;
@@ -259,8 +267,10 @@ private <X> void failed(@NotNull SQLConnection conn, @NotNull Promise<X> promise
     }
 
 
-    static class LegacyJooqxBuilderImpl extends SQLExecutorBuilderImpl<SQLClient, JsonArray, LegacySQLPreparedQuery,
-                                                                      ResultSet, LegacySQLCollector, LegacyJooqxBuilder>
+    @Deprecated
+    static class LegacyJooqxBuilderImpl extends
+                                        SQLExecutorBuilderImpl<SQLClient, JsonArray, LegacySQLPreparedQuery,
+                                                                  ResultSet, LegacySQLCollector, LegacyJooqxBuilder>
         implements LegacyJooqxBuilder {
 
         @Override
diff --git a/core/src/main/java/io/github/zero88/jooqx/LegacySQLPreparedQuery.java b/core/src/main/java/io/github/zero88/jooqx/LegacySQLPreparedQuery.java
index 4faf2a06..ea005d32 100644
--- a/core/src/main/java/io/github/zero88/jooqx/LegacySQLPreparedQuery.java
+++ b/core/src/main/java/io/github/zero88/jooqx/LegacySQLPreparedQuery.java
@@ -18,6 +18,7 @@
  * @since 1.0.0
  */
 @VertxGen
+@Deprecated
 public interface LegacySQLPreparedQuery extends SQLPreparedQuery<JsonArray> {
 
     /**