Skip to content

Commit 0cfc637

Browse files
committed
Merge branch '1.10_release_4.1.x' into hotfix_1.10_4.1.x_36409
# Conflicts: # core/src/main/java/com/dtstack/flink/sql/table/AbstractTableInfo.java
2 parents e052b0b + 6b75e1b commit 0cfc637

4 files changed

Lines changed: 33 additions & 14 deletions

File tree

localTest/pom.xml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -272,13 +272,7 @@
272272
<dependency>
273273
<groupId>com.alibaba</groupId>
274274
<artifactId>fastjson</artifactId>
275-
<version>1.2.29</version>
276-
<scope>compile</scope>
277-
</dependency>
278-
<dependency>
279-
<groupId>com.alibaba</groupId>
280-
<artifactId>fastjson</artifactId>
281-
<version>1.2.70</version>
275+
<version>1.2.76</version>
282276
<scope>compile</scope>
283277
</dependency>
284278

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@
9999
<dependency>
100100
<groupId>com.alibaba</groupId>
101101
<artifactId>fastjson</artifactId>
102-
<version>1.2.70</version>
102+
<version>1.2.76</version>
103103
</dependency>
104104
</dependencies>
105105

redis5/redis5-side/redis-async-side/pom.xml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,20 @@
3535
<plugin>
3636
<groupId>org.apache.maven.plugins</groupId>
3737
<artifactId>maven-shade-plugin</artifactId>
38-
<version>1.4</version>
38+
<version>3.0.0</version>
3939
<executions>
4040
<execution>
4141
<phase>package</phase>
4242
<goals>
4343
<goal>shade</goal>
4444
</goals>
4545
<configuration>
46+
<relocations>
47+
<relocation>
48+
<pattern>io.netty</pattern>
49+
<shadedPattern>com.dtstack.flink.sql.side.redis.io.netty</shadedPattern>
50+
</relocation>
51+
</relocations>
4652
<artifactSet>
4753
<excludes>
4854

redis5/redis5-side/redis-async-side/src/main/java/com/dtstack/flink/sql/side/redis/RedisAsyncReqRow.java

Lines changed: 24 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -37,17 +37,18 @@
3737
import io.lettuce.core.api.async.RedisKeyAsyncCommands;
3838
import io.lettuce.core.cluster.RedisClusterClient;
3939
import io.lettuce.core.cluster.api.StatefulRedisClusterConnection;
40+
import io.lettuce.core.internal.HostAndPort;
4041
import org.apache.commons.collections.MapUtils;
4142
import org.apache.commons.lang.StringUtils;
4243
import org.apache.flink.api.java.typeutils.RowTypeInfo;
4344
import org.apache.flink.configuration.Configuration;
4445
import org.apache.flink.streaming.api.functions.async.ResultFuture;
4546
import org.apache.flink.table.dataformat.BaseRow;
4647

48+
import java.util.ArrayList;
4749
import java.util.List;
4850
import java.util.Map;
4951
import java.util.Objects;
50-
import java.util.function.Consumer;
5152
import java.util.regex.Matcher;
5253

5354
/**
@@ -133,16 +134,34 @@ private void buildRedisClient(RedisSideTableInfo tableInfo){
133134
async = connection.async();
134135
break;
135136
case CLUSTER:
136-
RedisURI clusterURI = RedisURI.create("redis://" + url);
137-
clusterURI.setPassword(password);
138-
clusterClient = RedisClusterClient.create(clusterURI);
137+
List<RedisURI> clusterURIs = buildClusterURIs(url);
138+
clusterClient = RedisClusterClient.create(clusterURIs);
139139
clusterConnection = clusterClient.connect();
140140
async = clusterConnection.async();
141141
default:
142142
break;
143143
}
144144
}
145145

146+
private List<RedisURI> buildClusterURIs(String url) {
147+
String password = redisSideTableInfo.getPassword();
148+
String database = redisSideTableInfo.getDatabase();
149+
String[] addresses = StringUtils.split(url, ",");
150+
List<RedisURI> redisURIs = new ArrayList<>(addresses.length);
151+
for (String addr : addresses){
152+
HostAndPort hostAndPort = HostAndPort.parse(addr);
153+
RedisURI redisURI = RedisURI.create(hostAndPort.hostText, hostAndPort.port);
154+
if (StringUtils.isNotEmpty(password)) {
155+
redisURI.setPassword(password);
156+
}
157+
if (StringUtils.isNotEmpty(database)) {
158+
redisURI.setDatabase(Integer.valueOf(database));
159+
}
160+
redisURIs.add(redisURI);
161+
}
162+
return redisURIs;
163+
}
164+
146165
private RedisURI.Builder buildSentinelUri(
147166
String host,
148167
String port,
@@ -171,7 +190,7 @@ public void handleAsyncInvoke(Map<String, Object> inputParams, BaseRow input, Re
171190
if (MapUtils.isNotEmpty(values)) {
172191
try {
173192
BaseRow row = fillData(input, values);
174-
dealCacheData(key,CacheObj.buildCacheObj(ECacheContentType.SingleLine, row));
193+
dealCacheData(key, CacheObj.buildCacheObj(ECacheContentType.SingleLine, values));
175194
RowDataComplete.completeBaseRow(resultFuture, row);
176195
} catch (Exception e) {
177196
dealFillDataError(input, resultFuture, e);

0 commit comments

Comments
 (0)