Skip to content

Commit 8c9ac53

Browse files
committed
temporarily disable calling check_tw_to_close() in redis per flow to see if its the main source of latency
1 parent 5902fe3 commit 8c9ac53

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

slips_files/core/database/redis_db/profile_handler.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1577,8 +1577,9 @@ def mark_profile_tw_as_modified(self, profileid, twid, timestamp):
15771577
data = {f"{profileid}{self.separator}{twid}": float(timestamp)}
15781578
self.r.zadd(self.constants.MODIFIED_TIMEWINDOWS, data)
15791579
self.publish("tw_modified", f"{profileid}:{twid}")
1580-
# Check if we should close some TW
1581-
self.check_tw_to_close()
1580+
# # Check if we should close some TW
1581+
# self.check_tw_to_close() # @@@@@@@@@@@@@@@@@ TODO temporarily
1582+
# disabled
15821583

15831584
def publish_new_letter(
15841585
self, new_symbol: str, profileid: str, twid: str, tupleid: str, flow

slips_files/core/profiler_worker.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -285,6 +285,7 @@ def store_features_going_out(self, flow, profileid, twid):
285285

286286
# now that slips successfully parsed the flow,
287287
# mark this profile as modified
288+
# @@@@@@@@@@@@@ !! TODO this is the second suspect,
288289
self.db.mark_profile_tw_as_modified(profileid, twid, "")
289290
return True
290291

0 commit comments

Comments
 (0)