Skip to content

Commit 8f0b36d

Browse files
committed
[Add-more-logs] Trying to add more logs specifically for the scheduler and close
1 parent 6f45dc7 commit 8f0b36d

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

JShellAPI/src/main/java/org/togetherjava/jshellapi/service/JShellService.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,6 +206,7 @@ public String id() {
206206

207207
@Override
208208
public void close() {
209+
LOGGER.debug("Close called for session {}.", id);
209210
try {
210211
dockerService.killContainerByName(containerName());
211212
try {

JShellAPI/src/main/java/org/togetherjava/jshellapi/service/JShellSessionService.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ public class JShellSessionService {
2828
private void initScheduler() {
2929
scheduler = Executors.newSingleThreadScheduledExecutor();
3030
scheduler.scheduleAtFixedRate(() -> {
31+
LOGGER.info("Scheduler heartbeat: started.");
3132
jshellSessions.keySet()
3233
.stream()
3334
.filter(id -> jshellSessions.get(id).isClosed())
@@ -36,6 +37,7 @@ private void initScheduler() {
3637
.stream()
3738
.filter(id -> jshellSessions.get(id).shouldDie())
3839
.toList();
40+
LOGGER.info("Scheduler heartbeat: sessions ready to die: {}", toDie);
3941
for (String id : toDie) {
4042
try {
4143
deleteSession(id);
@@ -81,6 +83,7 @@ public JShellService oneTimeSession(@Nullable StartupScriptId startupScriptId)
8183
}
8284

8385
public void deleteSession(String id) throws DockerException {
86+
LOGGER.debug("Soft delete called for session {}.", id);
8487
JShellService service = jshellSessions.remove(id);
8588
service.stop();
8689
scheduler.schedule(service::close, 500, TimeUnit.MILLISECONDS);

0 commit comments

Comments
 (0)