Skip to content
This repository was archived by the owner on May 21, 2026. It is now read-only.

Dev #32

Merged
vitorhugo-java merged 7 commits into from
Feb 19, 2026
Merged

Dev #32

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
438762b
Stream JSON readers; change chunk log to debug
vitorhugo-java Feb 19, 2026
f9a37fc
Refactor TMDB data fetching to stream
vitorhugo-java Feb 19, 2026
05ade02
Fix media findById handling and update logic
vitorhugo-java Feb 19, 2026
7dcef56
Fix authorization role in stopJob method
vitorhugo-java Feb 19, 2026
ce9afd8
log debug
vitorhugo-java Feb 19, 2026
43d1e0c
drop indexes and create seq
vitorhugo-java Feb 19, 2026
f29c553
Merge branch 'master' into dev
vitorhugo-java Feb 19, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/main/java/com/espacogeek/geek/batch/DelayChunkListener.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ public DelayChunkListener(long delayInMillis) {
@Override
public void afterChunk(@NotNull ChunkContext context) {
try {
log.info("Chunk completed successfully. Applying delay of {}ms...", delayInMillis);
log.debug("Chunk completed successfully. Applying delay of {}ms...", delayInMillis);
Thread.sleep(delayInMillis);
} catch (InterruptedException e) {
log.error("Delay thread interrupted", e);
Expand Down
3 changes: 2 additions & 1 deletion src/main/java/com/espacogeek/geek/models/AlternativeTitleModel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,8 @@
@NoArgsConstructor
public class AlternativeTitleModel implements Serializable {
@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
@GeneratedValue(strategy = GenerationType.SEQUENCE, generator = "gen_alternative_titles")
@SequenceGenerator(name = "gen_alternative_titles", sequenceName = "seq_alternative_titles", allocationSize = 50)
Comment on lines +28 to +29
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AlternativeTitleModel now uses @SequenceGenerator but this file doesn't import jakarta.persistence.SequenceGenerator, so it won't compile. Add the missing import (or consolidate JPA imports).

Copilot uses AI. Check for mistakes.
@Column(name = "id_alternative_title")
private Integer id;

Expand Down
3 changes: 2 additions & 1 deletion src/main/java/com/espacogeek/geek/models/ExternalReferenceModel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,8 @@
@AllArgsConstructor
public class ExternalReferenceModel implements Serializable {
@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
@GeneratedValue(strategy = GenerationType.SEQUENCE, generator = "gen_externals_references")
@SequenceGenerator(name = "gen_externals_references", sequenceName = "seq_externals_references", allocationSize = 50)
@Column(name = "id_external_reference")
private Integer id;

Expand Down
3 changes: 2 additions & 1 deletion src/main/java/com/espacogeek/geek/models/MediaModel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,8 @@
@NoArgsConstructor
public class MediaModel implements Serializable {
@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
@GeneratedValue(strategy = GenerationType.SEQUENCE, generator = "gen_medias")
@SequenceGenerator(name = "gen_medias", sequenceName = "seq_medias", allocationSize = 50)
Comment on lines +42 to +43
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MediaModel now uses @SequenceGenerator but there is no jakarta.persistence.SequenceGenerator import in this file, so it won't compile. Add the missing import (or use jakarta.persistence.*).

Copilot uses AI. Check for mistakes.
@Column(name = "id_media")
private Integer id;

Expand Down
3 changes: 2 additions & 1 deletion src/main/java/com/espacogeek/geek/models/PeopleModel.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@
@NoArgsConstructor
public class PeopleModel implements Serializable {
@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
@GeneratedValue(strategy = GenerationType.SEQUENCE, generator = "gen_people")
@SequenceGenerator(name = "gen_people", sequenceName = "seq_people", allocationSize = 50))
Comment on lines +30 to +31
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PeopleModel now uses @SequenceGenerator but the class doesn't import jakarta.persistence.SequenceGenerator, so this won't compile. Add the missing import (or switch to a wildcard jakarta.persistence.* import consistent with other models).

Copilot uses AI. Check for mistakes.
@Column(name = "id_person")
private Integer id;

Expand Down
3 changes: 3 additions & 0 deletions src/main/resources/application.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,9 @@ spring.jpa.properties.hibernate.jdbc.lob.non_contextual_creation=true
spring.threads.virtual.enabled=true
spring.jpa.database-platform=org.hibernate.dialect.MariaDBDialect
spring.main.allow-bean-definition-overriding=true
spring.jpa.properties.hibernate.jdbc.batch_size=100
spring.jpa.properties.hibernate.order_inserts=true
spring.jpa.properties.hibernate.order_updates=true
# spring.devtools.add-properties=false

spring.batch.job.enabled=false
Expand Down
11 changes: 11 additions & 0 deletions src/main/resources/db/migration/V16__drop_indexes_create_seq.sql
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
CREATE SEQUENCE seq_medias START WITH 1 INCREMENT BY 50;
CREATE SEQUENCE seq_externals_references START WITH 1 INCREMENT BY 50;
CREATE SEQUENCE seq_alternative_titles START WITH 1 INCREMENT BY 50;
CREATE SEQUENCE seq_people START WITH 1 INCREMENT BY 50;

ALTER SEQUENCE seq_medias RESTART WITH 458830;
ALTER SEQUENCE seq_externals_references RESTART WITH 201324;
ALTER SEQUENCE seq_alternative_titles RESTART WITH 122752;
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seq_people is created but never restarted to a value above the current max people.id_person. If the table already contains rows, the next insert using GenerationType.SEQUENCE can collide with existing IDs. Add an ALTER SEQUENCE seq_people RESTART WITH ... (ideally derived from current MAX(id_person)+1).

Suggested change
ALTER SEQUENCE seq_alternative_titles RESTART WITH 122752;
ALTER SEQUENCE seq_alternative_titles RESTART WITH 122752;
ALTER SEQUENCE seq_people RESTART WITH 1000000;

Copilot uses AI. Check for mistakes.

Comment on lines +6 to +9
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The ALTER SEQUENCE ... RESTART WITH <hard-coded> values make this migration environment/data dependent. If IDs differ between environments (or future restores), this can either cause collisions or waste large ID ranges. Prefer computing MAX(id)+1 at migration time (e.g., via dynamic SQL) to make the migration deterministic.

Suggested change
ALTER SEQUENCE seq_medias RESTART WITH 458830;
ALTER SEQUENCE seq_externals_references RESTART WITH 201324;
ALTER SEQUENCE seq_alternative_titles RESTART WITH 122752;
DO $$
DECLARE
v_next_value bigint;
BEGIN
-- Set seq_medias based on current data
SELECT COALESCE(MAX(id), 0) + 1 INTO v_next_value FROM medias;
PERFORM setval('seq_medias', v_next_value, false);
-- Set seq_externals_references based on current data
SELECT COALESCE(MAX(id), 0) + 1 INTO v_next_value FROM externals_references;
PERFORM setval('seq_externals_references', v_next_value, false);
-- Set seq_alternative_titles based on current data
SELECT COALESCE(MAX(id), 0) + 1 INTO v_next_value FROM alternative_titles;
PERFORM setval('seq_alternative_titles', v_next_value, false);
END;
$$;

Copilot uses AI. Check for mistakes.
ALTER TABLE medias DROP INDEX idx_title;
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this migration, idx_title is dropped from medias, but the previous migration creates idx_title on alternative_titles (see V15). Dropping it from the wrong table will fail at runtime; drop idx_title from alternative_titles instead (and keep idx_name drop on medias if intended).

Suggested change
ALTER TABLE medias DROP INDEX idx_title;
ALTER TABLE alternative_titles DROP INDEX idx_title;

Copilot uses AI. Check for mistakes.
ALTER TABLE medias DROP INDEX idx_name;
Comment on lines +8 to +11
Copy link

Copilot AI Feb 19, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This migration drops idx_name/idx_title, but the corresponding JPA entities still declare these indexes via @Table(indexes=...) (MediaModel has idx_name, AlternativeTitleModel has idx_title). With spring.jpa.hibernate.ddl-auto=update, Hibernate may recreate the indexes on startup, effectively undoing this migration. If the goal is to permanently remove these indexes, also remove/adjust the @Index annotations (or set ddl-auto to validate).

Suggested change
ALTER SEQUENCE seq_alternative_titles RESTART WITH 122752;
ALTER TABLE medias DROP INDEX idx_title;
ALTER TABLE medias DROP INDEX idx_name;
ALTER SEQUENCE seq_alternative_titles RESTART WITH 122752;

Copilot uses AI. Check for mistakes.
Loading