Skip to content

Commit 75618d5

Browse files
committed
Analytics update
- Metrics: renaming persist() to processEvent(); - resources/db: V16 update, removing default value for happened_at column
1 parent a27f89e commit 75618d5

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

application/src/main/java/org/togetherjava/tjbot/features/analytics/Metrics.java

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
import org.slf4j.LoggerFactory;
55

66
import org.togetherjava.tjbot.db.Database;
7-
import org.togetherjava.tjbot.db.generated.tables.Analytics;
7+
import org.togetherjava.tjbot.db.generated.tables.MetricEvents;
88

99
import java.time.Instant;
1010
import java.util.concurrent.ExecutorService;
@@ -37,7 +37,7 @@ public Metrics(Database database) {
3737
public void count(String event) {
3838
logger.debug("Counting new record for event: {}", event);
3939
Instant moment = Instant.now();
40-
service.submit(() -> persist(event, moment));
40+
service.submit(() -> processEvent(event, moment));
4141

4242
}
4343

@@ -46,13 +46,11 @@ public void count(String event) {
4646
* @param event the event to save
4747
* @param happenedAt the moment when the event is dispatched
4848
*/
49-
private void persist(String event, Instant happenedAt) {
50-
logger.debug("Persisting event: {}, at {}", event, happenedAt);
51-
database.write(context -> context.newRecord(Analytics.ANALYTICS)
49+
private void processEvent(String event, Instant happenedAt) {
50+
database.write(context -> context.newRecord(MetricEvents.METRIC_EVENTS)
5251
.setEvent(event)
5352
.setHappenedAt(happenedAt)
5453
.insert());
55-
logger.debug("Event {} persisted successfully", event);
5654
}
5755

5856
}
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
CREATE TABLE analytics
1+
CREATE TABLE metric_events
22
(
33
id INTEGER PRIMARY KEY AUTOINCREMENT,
44
event TEXT NOT NULL,
5-
happened_at TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP
5+
happened_at TIMESTAMP NOT NULL
66
);

0 commit comments

Comments
 (0)