xuxiaowei-com-cn commented on code in PR #8054:
URL: https://github.com/apache/incubator-seata/pull/8054#discussion_r3159912559
##########
server/src/main/java/org/apache/seata/server/storage/redis/store/RedisTransactionStoreManager.java:
##########
@@ -500,7 +499,7 @@ public List<GlobalSession>
readSortByTimeoutBeginSessions(boolean withBranchSess
// queryCount
final long queryCount = Math.min(logQueryLimit, countGlobalSessions);
try (Jedis jedis = JedisPooledFactory.getJedisInstance()) {
- Set<String> values = jedis.zrangeByScore(
+ List<String> values = jedis.zrangeByScore(
REDIS_SEATA_BEGIN_TRANSACTIONS_KEY, 0,
System.currentTimeMillis(), 0, (int) queryCount);
List<Map<String, String>> rep;
Review Comment:
# English
1. `org.apache.seata:seata-dependencies` uses
`<jedis.version>3.8.0</jedis.version>` to control the version of jedis.
`org.apache.seata:seata-server` inherits from `org.apache.seata:seata-parent`,
and `org.apache.seata:seata-parent` introduces
`org.apache.seata:seata-dependencies`.
2. `org.springframework.boot:spring-boot-dependencies:4.x` uses
`<jedis.version>7.0.0</jedis.version>` to control the version of jedis.
`org.apache.seata:seata-server` introduces
`org.springframework.boot:spring-boot-dependencies`.
3. According to the dependency order above, `org.apache.seata:seata-server`
uses **7.0.0** instead of **3.8.0**.
# 中文
1. `org.apache.seata:seata-dependencies` 中使用
`<jedis.version>3.8.0</jedis.version>` 控制 jedis
的版本。`org.apache.seata:seata-server` 继承了
`org.apache.seata:seata-parent`，`org.apache.seata:seata-parent` 引入了
`org.apache.seata:seata-dependencies`。
2. `org.springframework.boot:spring-boot-dependencies:4.x` 中使用
`<jedis.version>7.0.0</jedis.version>` 控制 jedis
的版本。`org.apache.seata:seata-server` 引入了
`org.springframework.boot:spring-boot-dependencies`
3. 根据上述依赖的先后顺序，`org.apache.seata:seata-server` 使用的是 **7.0.0**，而不是 **3.8.0**。
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
