Skip to content

Commit f84cc5a

Browse files
committed
fix db retrieve and add more logging
1 parent 8bb41f7 commit f84cc5a

2 files changed

Lines changed: 8 additions & 6 deletions

File tree

src/main/java/de/mediathekview/mserver/base/utils/FilmDBService.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
import de.mediathekview.mserver.daten.Filmlist;
88
import de.mediathekview.mserver.daten.GsonDurationAdapter;
99
import de.mediathekview.mserver.daten.GsonLocalDateTimeAdapter;
10+
import de.mediathekview.mserver.daten.Sender;
1011

1112
import javax.sql.DataSource;
1213

@@ -106,16 +107,16 @@ public void deleteFilms(Collection<Film> abandonedFilmlist) {
106107
/////////////////////////////////////////////////////////////////////////////////////////
107108

108109
public Optional<Filmlist> readFilmlistFromDB() {
109-
return readFilmlistFromDB("");
110+
return readFilmlistFromDB("", "");
110111
}
111112

112-
public Optional<Filmlist> readFilmlistFromDB(String where) {
113+
public Optional<Filmlist> readFilmlistFromDB(String where, String limit) {
113114
long start = System.currentTimeMillis();
114115
LOG.debug("import filmlist from DB");
115116
int readCounter = 0;
116117
Filmlist list = new Filmlist();
117118
try (Connection con = dataSource.getConnection();
118-
PreparedStatement ps = con.prepareStatement("SELECT data FROM filme " + where + " ORDER BY data ->> 'sender', data ->> 'thema', data ->> 'titel'");
119+
PreparedStatement ps = con.prepareStatement("SELECT data FROM filme " + where + " ORDER BY data ->> 'sender', data ->> 'thema', data ->> 'titel' " + limit);
119120
) {
120121
ps.setFetchSize(50000);
121122
try (ResultSet rs = ps.executeQuery()) {
@@ -136,7 +137,7 @@ public Optional<Filmlist> readFilmlistFromDB(String where) {
136137
/////////////////////////////////////////////////////////////////////////////////////////
137138
/////////////////////////////////////////////////////////////////////////////////////////
138139

139-
public <T> List<T> filterNewVideos(List<T> videos, Function<T, String> idExtractor) {
140+
public <T> List<T> filterNewVideos(Sender sender, List<T> videos, Function<T, String> idExtractor) {
140141
if(!PostgreSQLDataSourceProvider.isEnabled()) {
141142
return videos;
142143
}
@@ -183,7 +184,7 @@ public <T> List<T> filterNewVideos(List<T> videos, Function<T, String> idExtract
183184
for (Future<List<T>> f : futures) {
184185
result.addAll(f.get());
185186
}
186-
LOG.debug("Filtered {} (in {} out {})",(videos.size()-result.size()), videos.size(), result.size());
187+
LOG.debug("Filtered {} in {} (in {} vs out {})",(videos.size()-result.size()), sender.getName(), videos.size(), result.size());
187188
return result;
188189
} catch (Exception e) {
189190
LOG.error("{}", e);

src/main/java/de/mediathekview/mserver/crawler/basic/AbstractCrawler.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,11 @@ protected AbstractCrawler(
7878
public <T> Queue<T> filterExistingFilms(Collection<T> input, Function<T, String> idExtractor) {
7979
return new ArrayDeque<>(
8080
filmDBService.filterNewVideos(
81+
getSender(),
8182
new ArrayList<>(input),
8283
idExtractor
8384
)
84-
);
85+
);
8586
}
8687

8788
@Override

0 commit comments

Comments
 (0)