Skip to content
This repository was archived by the owner on Oct 3, 2025. It is now read-only.

Feature/performant old filmlist reader #150

Open
Nicklas2751 wants to merge 13 commits into
base: develop
Choose a base branch
from
Open

Feature/performant old filmlist reader #150

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
191dd74
Add spring parent pom, cleanup dependencies, update deps, add lombok …
Nicklas2751 Feb 6, 2022
1558bc1
Add a more performant old filmlist format reader
Nicklas2751 Feb 6, 2022
49b3861
Fix a maven pom waring with empty xml element
Nicklas2751 Feb 6, 2022
5977f74
Fix missing plugin version
Nicklas2751 Feb 6, 2022
2008975
Fix code smells
Nicklas2751 Feb 6, 2022
8f475fe
Move plugin versions to vars, Remove unused RawFilm fields & change c…
Nicklas2751 Feb 19, 2022
afc6fe9
Change entry delimiter pattern because it may contain whitespace
Nicklas2751 Feb 20, 2022
0a6298b
Add missing test for raw film to film mapper
Nicklas2751 Feb 25, 2022
e90f29a
Refactor raw film to film mapper
Nicklas2751 Feb 25, 2022
6901d68
Don't know if it works but here is our pairing
TheSasch Mar 5, 2022
766e558
Remove mock & add more tests
Nicklas2751 Mar 5, 2022
f6dfac0
Add more mapper tests
Nicklas2751 Mar 6, 2022
a1ad4df
Fix small mapper bugs & add more mapper tests
Nicklas2751 Mar 9, 2022
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
29 changes: 29 additions & 0 deletions .run/MLib [Sort pom].run.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
<component name="ProjectRunConfigurationManager">
<configuration default="false" name="MLib [Sort pom]" type="MavenRunConfiguration" factoryName="Maven">
<MavenSettings>
<option name="myGeneralSettings" />
<option name="myRunnerSettings" />
<option name="myRunnerParameters">
<MavenRunnerParameters>
<option name="profiles">
<set />
</option>
<option name="goals">
<list>
<option value="com.github.ekryd.sortpom:sortpom-maven-plugin:sort" />
<option value="-Dsort.keepBlankLines" />
<option value="-Dsort.predefinedSortOrder=recommended_2008_06" />
</list>
</option>
<option name="pomFileName" />
<option name="profilesMap">
<map />
</option>
<option name="resolveToWorkspace" value="false" />
<option name="workingDirPath" value="$PROJECT_DIR$" />
</MavenRunnerParameters>
</option>
</MavenSettings>
<method v="2" />
</configuration>
</component>
127 changes: 56 additions & 71 deletions pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,13 @@
<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
<modelVersion>4.0.0</modelVersion>
<parent>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-starter-parent</artifactId>

@pidoubleyou pidoubleyou Feb 9, 2022

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

wieso Spring Boot? wird doch in MLib gar nicht verwendet.

@Nicklas2751 Nicklas2751 Feb 19, 2022

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Im spring Boot parent sind für viele Abhängigkeiten die Versionen aufeinander abgestimmt und getestet. Wenn wir den, wie hier von mir getan, nutzen müssen wir uns nicht um Kompatibilität einzelner Versionen zueinander kümmern. Zum updaten von dependencies können wir so auch "einfach" die Spring version erhöhen und müssen uns dann nur noch um die zusätzlichen Deps kümmern.

<version>2.6.3</version>
<relativePath/>
<!-- lookup parent from repository -->
</parent>

<groupId>de.mediathekview</groupId>
<artifactId>MLib</artifactId>
Expand Down Expand Up @@ -69,62 +76,18 @@
<maven.compiler.source>17</maven.compiler.source>
<maven.compiler.target>17</maven.compiler.target>

<assertj.version>3.21.0</assertj.version>
<commons-compress.version>1.21</commons-compress.version>
<commons-text.version>1.9</commons-text.version>
<gson.version>2.8.9</gson.version>
<jakarta-activation.version>2.0.1</jakarta-activation.version>
<javax-jaxb.version>2.4.0-b180830.0359</javax-jaxb.version>
<jetbrains-annotations.version>22.0.0</jetbrains-annotations.version>
<jersey.version>3.0.3</jersey.version>
<junit-jupiter.version>5.7.2</junit-jupiter.version>
<log4j2.version>2.17.1</log4j2.version>
<maven-clean-plugin.version>3.1.0</maven-clean-plugin.version>
<maven-compiler-plugin.version>3.8.1</maven-compiler-plugin.version>
<maven-deploy-plugin.version>2.8.2</maven-deploy-plugin.version>
<maven-gpg-plugin.version>1.6</maven-gpg-plugin.version>
<maven-install-plugin.version>2.5.2</maven-install-plugin.version>
<maven-javadoc-plugin.version>3.1.1</maven-javadoc-plugin.version>
<maven-jar-plugin.version>3.2.0</maven-jar-plugin.version>
<maven-project-info-reports-plugin.version>3.0.0</maven-project-info-reports-plugin.version>
<maven-resources-plugin.version>3.2.0</maven-resources-plugin.version>
<maven-source-plugin.version>3.2.1</maven-source-plugin.version>
<maven-surefire-plugin.version>2.22.2</maven-surefire-plugin.version>
<maven-site-plugin.version>3.8.2</maven-site-plugin.version>
<mockito.version>4.2.0</mockito.version>
<nexus-staging-maven-plugin.version>1.6.8</nexus-staging-maven-plugin.version>
<okhttp3.version>4.9.3</okhttp3.version>
<wiremock.version>2.27.2</wiremock.version>
<wiremock.version>2.32.0</wiremock.version>
<yacl4j-core.version>0.9.2</yacl4j-core.version>
<xz.version>1.9</xz.version>
<mapstruct.version>1.4.2.Final</mapstruct.version>
<lombok-mapstruct-binding.version>0.2.0</lombok-mapstruct-binding.version>
<nexus-staging-maven-plugin.version>1.6.8</nexus-staging-maven-plugin.version>
<maven-gpg-plugin.version>3.0.1</maven-gpg-plugin.version>
</properties>

<dependencyManagement>
<dependencies>
<dependency>
<groupId>org.glassfish.jersey</groupId>
<artifactId>jersey-bom</artifactId>
<version>${jersey.version}</version>
<type>pom</type>
<scope>import</scope>
</dependency>
<dependency>
<groupId>org.apache.logging.log4j</groupId>
<artifactId>log4j-bom</artifactId>
<version>${log4j2.version}</version>
<type>pom</type>
<scope>import</scope>
</dependency>
<dependency>
<groupId>org.junit</groupId>
<artifactId>junit-bom</artifactId>
<version>${junit-jupiter.version}</version>
<type>pom</type>
<scope>import</scope>
</dependency>

</dependencies>
</dependencyManagement>
<dependencies>
<dependency>
<groupId>org.apache.commons</groupId>
Expand Down Expand Up @@ -185,10 +148,18 @@
</dependency>

<!-- Logging -->
<dependency>
<groupId>org.apache.logging.log4j</groupId>
<artifactId>log4j-api</artifactId>
</dependency>
<dependency>
<groupId>org.apache.logging.log4j</groupId>
<artifactId>log4j-core</artifactId>
</dependency>
<dependency>
<groupId>org.apache.logging.log4j</groupId>
<artifactId>log4j-slf4j-impl</artifactId>
</dependency>

<!-- JUnit 5 Jupiter -->
<dependency>
Expand Down Expand Up @@ -235,11 +206,22 @@
<!-- Wiremock HTTP server mock for integration tests -->
<dependency>
<groupId>com.github.tomakehurst</groupId>
<artifactId>wiremock</artifactId>
<artifactId>wiremock-jre8</artifactId>
<version>${wiremock.version}</version>
<scope>test</scope>
</dependency>

<dependency>
<groupId>org.mapstruct</groupId>
<artifactId>mapstruct</artifactId>
<version>${mapstruct.version}</version>
</dependency>
<dependency>
<groupId>org.projectlombok</groupId>
<artifactId>lombok</artifactId>
<optional>true</optional>
</dependency>

</dependencies>

<build>
Expand All @@ -266,13 +248,13 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-site-plugin</artifactId>
<version>${maven-site-plugin.version}</version>
<version>3.8.2</version>
Comment thread
Nicklas2751 marked this conversation as resolved.
Outdated
</plugin>

<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-project-info-reports-plugin</artifactId>
<version>${maven-project-info-reports-plugin.version}</version>
<version>3.0.0</version>
Comment thread
Nicklas2751 marked this conversation as resolved.
Outdated
</plugin>

<!-- Build a source Jar -->
Expand Down Expand Up @@ -305,24 +287,28 @@
</executions>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-compiler-plugin</artifactId>
<version>${maven-compiler-plugin.version}</version>
</plugin>
<plugin>
<artifactId>maven-surefire-plugin</artifactId>
<version>${maven-surefire-plugin.version}</version>
</plugin>
<plugin>
<artifactId>maven-jar-plugin</artifactId>
<version>${maven-jar-plugin.version}</version>
</plugin>
<plugin>
<artifactId>maven-install-plugin</artifactId>
<version>${maven-install-plugin.version}</version>
</plugin>
<plugin>
<artifactId>maven-deploy-plugin</artifactId>
<version>${maven-deploy-plugin.version}</version>
<configuration>
<annotationProcessorPaths>
<path>
<groupId>org.mapstruct</groupId>
<artifactId>mapstruct-processor</artifactId>
<version>${mapstruct.version}</version>
</path>
<path>
<groupId>org.projectlombok</groupId>
<artifactId>lombok</artifactId>
<version>${lombok.version}</version>
</path>
<dependency>
<groupId>org.projectlombok</groupId>
<artifactId>lombok-mapstruct-binding</artifactId>
<version>${lombok-mapstruct-binding.version}</version>
</dependency>
</annotationProcessorPaths>
</configuration>
</plugin>

</plugins>
Expand All @@ -334,7 +320,6 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-javadoc-plugin</artifactId>
<version>${maven-javadoc-plugin.version}</version>
<configuration>
<show>private</show>
<nohelp>true</nohelp>
Expand All @@ -352,7 +337,7 @@
<plugin>
<groupId>org.sonatype.plugins</groupId>
<artifactId>nexus-staging-maven-plugin</artifactId>
<version>${nexus-staging-maven-plugin.version}</version>
<version>1.6.8</version>
Comment thread
Nicklas2751 marked this conversation as resolved.
<extensions>true</extensions>
<configuration>
<serverId>ossrh</serverId>
Expand All @@ -363,7 +348,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-gpg-plugin</artifactId>
<version>${maven-gpg-plugin.version}</version>
<version>3.0.1</version>
Comment thread
Nicklas2751 marked this conversation as resolved.
<executions>
<execution>
<id>sign-artifacts</id>
Expand Down
9 changes: 0 additions & 9 deletions src/main/java/de/mediathekview/mlib/filmlisten/reader/CantReadFilmException.java
View file
Open in desktop

This file was deleted.

Loading