You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ri...@apache.org on 2021/11/09 18:40:29 UTC
[geode] branch GEM-3421-Geode-for-Redis-String-TLS-Tests updated
(206d15d -> c676c26)
This is an automated email from the ASF dual-hosted git repository.
ringles pushed a change to branch GEM-3421-Geode-for-Redis-String-TLS-Tests
in repository https://gitbox.apache.org/repos/asf/geode.git.
from 206d15d one more log bump
new 6506051 more logging and bumped SHA
new c676c26 Log Jedis and Spring stuff too
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.../org/apache/geode/redis/internal/executor/key/KeysExecutor.java | 4 ++++
.../org/apache/geode/redis/internal/executor/key/RenameExecutor.java | 4 ++++
geode-log4j/src/main/resources/log4j2.xml | 5 ++++-
3 files changed, 12 insertions(+), 1 deletion(-)
[geode] 01/02: more logging and bumped SHA
Posted by ri...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
ringles pushed a commit to branch GEM-3421-Geode-for-Redis-String-TLS-Tests
in repository https://gitbox.apache.org/repos/asf/geode.git
commit 6506051869323a319931c5821facde78540da72c
Author: Ray Ingles <ri...@vmware.com>
AuthorDate: Fri Nov 5 16:35:04 2021 -0400
more logging and bumped SHA
---
.../org/apache/geode/redis/internal/executor/key/KeysExecutor.java | 1 +
.../org/apache/geode/redis/internal/executor/key/RenameExecutor.java | 4 ++++
2 files changed, 5 insertions(+)
diff --git a/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/KeysExecutor.java b/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/KeysExecutor.java
index 9777c2d..a7a3555 100755
--- a/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/KeysExecutor.java
+++ b/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/KeysExecutor.java
@@ -36,6 +36,7 @@ public class KeysExecutor implements CommandExecutor {
@Override
public RedisResponse executeCommand(Command command, ExecutionHandlerContext context) {
List<byte[]> commandElems = command.getProcessedCommand();
+ logger.error("KEYS, args are:" + commandElems);
byte[] glob = commandElems.get(1);
Set<RedisKey> allKeys = context.getRegion().keySet();
List<byte[]> matchingKeys = new ArrayList<>();
diff --git a/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/RenameExecutor.java b/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/RenameExecutor.java
index 16fb23c..fba6134 100644
--- a/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/RenameExecutor.java
+++ b/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/RenameExecutor.java
@@ -16,16 +16,20 @@
package org.apache.geode.redis.internal.executor.key;
+import org.apache.logging.log4j.Logger;
+import org.apache.geode.logging.internal.log4j.api.LogService;
import org.apache.geode.redis.internal.data.RedisKey;
import org.apache.geode.redis.internal.executor.RedisResponse;
import org.apache.geode.redis.internal.netty.ExecutionHandlerContext;
public class RenameExecutor extends AbstractRenameExecutor {
+ private static final Logger logger = LogService.getLogger();
@Override
protected boolean executeRenameCommand(RedisKey key, RedisKey newKey,
ExecutionHandlerContext context) {
+ logger.error("RENAME, args are:" + key + " " + newKey);
return rename(context, key, newKey, false);
}
[geode] 02/02: Log Jedis and Spring stuff too
Posted by ri...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
ringles pushed a commit to branch GEM-3421-Geode-for-Redis-String-TLS-Tests
in repository https://gitbox.apache.org/repos/asf/geode.git
commit c676c2692462e1ad0c780b184537a17b379d6c1b
Author: Ray Ingles <ri...@vmware.com>
AuthorDate: Tue Nov 9 13:39:23 2021 -0500
Log Jedis and Spring stuff too
---
.../org/apache/geode/redis/internal/executor/key/KeysExecutor.java | 3 +++
geode-log4j/src/main/resources/log4j2.xml | 5 ++++-
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/KeysExecutor.java b/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/KeysExecutor.java
index a7a3555..c8a686b 100755
--- a/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/KeysExecutor.java
+++ b/geode-for-redis/src/main/java/org/apache/geode/redis/internal/executor/key/KeysExecutor.java
@@ -42,13 +42,16 @@ public class KeysExecutor implements CommandExecutor {
List<byte[]> matchingKeys = new ArrayList<>();
GlobPattern pattern = new GlobPattern(glob);
+ logger.error("KEYS, glob is:" + pattern);
for (RedisKey key : allKeys) {
+ logger.error("KEYS, cur key is:" + key);
byte[] keyBytes = key.toBytes();
if (pattern.matches(keyBytes)) {
matchingKeys.add(keyBytes);
}
}
+ logger.error("KEYS, matchingKeys length:" + matchingKeys.size());
return RedisResponse.array(matchingKeys, true);
}
diff --git a/geode-log4j/src/main/resources/log4j2.xml b/geode-log4j/src/main/resources/log4j2.xml
index 9d266bc..7189404 100755
--- a/geode-log4j/src/main/resources/log4j2.xml
+++ b/geode-log4j/src/main/resources/log4j2.xml
@@ -32,7 +32,10 @@
</Appenders>
<Loggers>
<Logger name="com.gemstone" level="INFO" additivity="true"/>
- <Logger name="org.apache.geode" level="INFO" additivity="true">
+ <Logger name="io.netty" level="DEBUG" additivity="true"/>
+ <Logger name="redis.clients.jedis" level="DEBUG" additivity="true"/>
+ <Logger name="org.springframework" level="DEBUG" additivity="true"/>
+ <Logger name="org.apache.geode" level="DEBUG" additivity="true">
<filters>
<MarkerFilter marker="GEODE_VERBOSE" onMatch="DENY" onMismatch="NEUTRAL"/>
</filters>