kengotoda/brownie

A distributed video converter

Kengo TODA (eller86)

A hobby programmer from 15 years old. 😋

Features

• Maintain apps with TDD
  • Java, JUnit Jupiter, TypeScript, Jest, Playwright, Selenide, etc.
• Customize and optimize Development Workflows
  • GitHub Actions, Jenkins, Travis CI, Ant, Maven, Gradle, npm, yarn, Sphinx, Docker, etc.
  • Develop extensions for Ant, Maven, Gradle, Jenkins, GitHub Probot, and GitHub Action.
• Static Code Analysis and Bytecode Manipulation
  • Work for SpotBugs the spiritual successor of FindBugs.
  • Develop plugins for SpotBugs, Google Errorprone, SonarQube, PMD, etc. Also develop small tools with ObjectWeb ASM.
• Problem Solving, Leadership, Team/Project Management
  • About my business career, visit latest resumes or contact me at linkedin.

Contacts

Email to [email protected], or DM at twitter.

brownie's Issues

Cannot delete a file from admin console

FrontServer may have a problem, it cannot delete a file at #deleteFile(RoutingContext, String). We should use Single#create(onSubscribe) like RxHelper#get(HttpClient, String).

java.lang.IllegalStateException: You must set an handler for the HttpClientResponse before connecting
at io.vertx.core.http.impl.HttpClientRequestImpl.checkResponseHandler(HttpClientRequestImpl.java:763)

Try to replace com.eaio.uuid with JUG

If it can reduce size of .jar file, it would be better to replace.

• http://mediashelf.github.io/uuid-datepath-idmapper/generate_uuids.html
• http://repo1.maven.org/maven2/com/fasterxml/uuid/java-uuid-generator/3.1.4/

Refactoring

• Remove all Observable prefixes in class name
• Replace key with id
• Rename Task
• Use Single instead of Observable if returned entity should be 0 or 1
• Add javadoc to all classes/interfaces
• Stop using spring-boot
• Separate BackendServer to ThumbnailServer, EncodeServer and so on.

Introduce Circuit Breaker Pattern

If we stop using --link, application server may be launched before datastore becomes ready.
To support this case, it is better to introduce Circuit Breaker Pattern to data access layers.

Stop `--link consul`

--link is useful but it is not good for scaling; Docker Compose will deploy all linked nodes onto the same host.

Big problem is that, we use consul as DNS so almost all servers need to know where consul is.

Eliminate 'hidden' dependencies between microservices via common module

Currently we have Event class in common, it may force event consumer microservice to upgrade its common module when event publisher upgrades format of event. This means that event consumer microservice lacks control of its dependency.

Instead, we can use common class like JsonObject as event class. Then we can eliminate these dependencies.
It is also good to call multiple versions of REST API; one microservice may call multiple versions of one API, and if we use class, we need multiple event classes like FooEventV1 & FooEventV2.

Use API version as URL prefix (e.g. /v1.0/file/:fileId)

Enhancement around postgres

• Add ORDER BY to sort results of SELECT
• Ensure that we can use INDEX to SELECT