From 388810c043f129afb06e2537697107bd1b641a77 Mon Sep 17 00:00:00 2001 From: Sayyed-2727 Date: Wed, 29 Jul 2026 14:57:14 +0200 Subject: [PATCH] Implement similar products endpoint with timeout and partial-failure handling --- .gitattributes | 2 + .gitignore | 33 ++ .graphify/.building | 1 + .graphifyignore | 60 ++++ .mvn/wrapper/maven-wrapper.properties | 3 + mvnw | 295 ++++++++++++++++++ pom.xml | 113 +++++++ .../TechnicalAssessmentApplication.java | 13 + .../client/ProductApiClient.java | 72 +++++ .../config/RestClientConfig.java | 25 ++ .../controller/ProductController.java | 26 ++ .../exception/ExternalApiException.java | 12 + .../exception/GlobalExceptionHandler.java | 22 ++ .../exception/ProductNotFoundException.java | 8 + .../model/ProductDetail.java | 11 + .../service/SimilarProductService.java | 44 +++ src/main/resources/application.properties | 4 + 17 files changed, 744 insertions(+) create mode 100644 .gitattributes create mode 100644 .gitignore create mode 100644 .graphify/.building create mode 100644 .graphifyignore create mode 100644 .mvn/wrapper/maven-wrapper.properties create mode 100644 mvnw create mode 100644 pom.xml create mode 100644 src/main/java/com/example/TechnicalAssessment/TechnicalAssessmentApplication.java create mode 100644 src/main/java/com/example/TechnicalAssessment/client/ProductApiClient.java create mode 100644 src/main/java/com/example/TechnicalAssessment/config/RestClientConfig.java create mode 100644 src/main/java/com/example/TechnicalAssessment/controller/ProductController.java create mode 100644 src/main/java/com/example/TechnicalAssessment/exception/ExternalApiException.java create mode 100644 src/main/java/com/example/TechnicalAssessment/exception/GlobalExceptionHandler.java create mode 100644 src/main/java/com/example/TechnicalAssessment/exception/ProductNotFoundException.java create mode 100644 src/main/java/com/example/TechnicalAssessment/model/ProductDetail.java create mode 100644 src/main/java/com/example/TechnicalAssessment/service/SimilarProductService.java create mode 100644 src/main/resources/application.properties diff --git a/.gitattributes b/.gitattributes new file mode 100644 index 00000000..3b41682a --- /dev/null +++ b/.gitattributes @@ -0,0 +1,2 @@ +/mvnw text eol=lf +*.cmd text eol=crlf diff --git a/.gitignore b/.gitignore new file mode 100644 index 00000000..667aaef0 --- /dev/null +++ b/.gitignore @@ -0,0 +1,33 @@ +HELP.md +target/ +.mvn/wrapper/maven-wrapper.jar +!**/src/main/**/target/ +!**/src/test/**/target/ + +### STS ### +.apt_generated +.classpath +.factorypath +.project +.settings +.springBeans +.sts4-cache + +### IntelliJ IDEA ### +.idea +*.iws +*.iml +*.ipr + +### NetBeans ### +/nbproject/private/ +/nbbuild/ +/dist/ +/nbdist/ +/.nb-gradle/ +build/ +!**/src/main/**/build/ +!**/src/test/**/build/ + +### VS Code ### +.vscode/ diff --git a/.graphify/.building b/.graphify/.building new file mode 100644 index 00000000..e64e96e6 --- /dev/null +++ b/.graphify/.building @@ -0,0 +1 @@ +2026-07-29T12:01:37.288Z \ No newline at end of file diff --git a/.graphifyignore b/.graphifyignore new file mode 100644 index 00000000..584fae5e --- /dev/null +++ b/.graphifyignore @@ -0,0 +1,60 @@ +# axet-graphify-managed v3 +# Semantic files excluded — graphify runs in code-only AST mode +# Text documents (classified as semantic by graphify) +*.md +*.mdx +*.qmd +*.txt +*.rst +*.adoc +*.html +*.htm +*.yaml +*.yml +# Office / PDF / Google Workspace documents +*.pdf +*.docx +*.doc +*.pptx +*.ppt +*.xlsx +*.xls +*.odt +*.odp +*.ods +*.rtf +*.gdoc +*.gsheet +*.gslides +# Images +*.png +*.jpg +*.jpeg +*.gif +*.svg +*.webp +*.ico +*.bmp +*.tiff +# Audio / Video +*.mp4 +*.mp3 +*.wav +*.avi +*.mov +*.mkv +*.webm +*.m4a +*.m4v +*.ogg +# Common non-code dirs +node_modules/ +.git/ +dist/ +out/ +build/ +coverage/ +# Machine-generated source (huge files, noise in the graph, heavy to parse) +generated/ +__generated__/ +.graphify/ diff --git a/.mvn/wrapper/maven-wrapper.properties b/.mvn/wrapper/maven-wrapper.properties new file mode 100644 index 00000000..216df058 --- /dev/null +++ b/.mvn/wrapper/maven-wrapper.properties @@ -0,0 +1,3 @@ +wrapperVersion=3.3.4 +distributionType=only-script +distributionUrl=https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.9.16/apache-maven-3.9.16-bin.zip diff --git a/mvnw b/mvnw new file mode 100644 index 00000000..bd8896bf --- /dev/null +++ b/mvnw @@ -0,0 +1,295 @@ +#!/bin/sh +# ---------------------------------------------------------------------------- +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. +# ---------------------------------------------------------------------------- + +# ---------------------------------------------------------------------------- +# Apache Maven Wrapper startup batch script, version 3.3.4 +# +# Optional ENV vars +# ----------------- +# JAVA_HOME - location of a JDK home dir, required when download maven via java source +# MVNW_REPOURL - repo url base for downloading maven distribution +# MVNW_USERNAME/MVNW_PASSWORD - user and password for downloading maven +# MVNW_VERBOSE - true: enable verbose log; debug: trace the mvnw script; others: silence the output +# ---------------------------------------------------------------------------- + +set -euf +[ "${MVNW_VERBOSE-}" != debug ] || set -x + +# OS specific support. +native_path() { printf %s\\n "$1"; } +case "$(uname)" in +CYGWIN* | MINGW*) + [ -z "${JAVA_HOME-}" ] || JAVA_HOME="$(cygpath --unix "$JAVA_HOME")" + native_path() { cygpath --path --windows "$1"; } + ;; +esac + +# set JAVACMD and JAVACCMD +set_java_home() { + # For Cygwin and MinGW, ensure paths are in Unix format before anything is touched + if [ -n "${JAVA_HOME-}" ]; then + if [ -x "$JAVA_HOME/jre/sh/java" ]; then + # IBM's JDK on AIX uses strange locations for the executables + JAVACMD="$JAVA_HOME/jre/sh/java" + JAVACCMD="$JAVA_HOME/jre/sh/javac" + else + JAVACMD="$JAVA_HOME/bin/java" + JAVACCMD="$JAVA_HOME/bin/javac" + + if [ ! -x "$JAVACMD" ] || [ ! -x "$JAVACCMD" ]; then + echo "The JAVA_HOME environment variable is not defined correctly, so mvnw cannot run." >&2 + echo "JAVA_HOME is set to \"$JAVA_HOME\", but \"\$JAVA_HOME/bin/java\" or \"\$JAVA_HOME/bin/javac\" does not exist." >&2 + return 1 + fi + fi + else + JAVACMD="$( + 'set' +e + 'unset' -f command 2>/dev/null + 'command' -v java + )" || : + JAVACCMD="$( + 'set' +e + 'unset' -f command 2>/dev/null + 'command' -v javac + )" || : + + if [ ! -x "${JAVACMD-}" ] || [ ! -x "${JAVACCMD-}" ]; then + echo "The java/javac command does not exist in PATH nor is JAVA_HOME set, so mvnw cannot run." >&2 + return 1 + fi + fi +} + +# hash string like Java String::hashCode +hash_string() { + str="${1:-}" h=0 + while [ -n "$str" ]; do + char="${str%"${str#?}"}" + h=$(((h * 31 + $(LC_CTYPE=C printf %d "'$char")) % 4294967296)) + str="${str#?}" + done + printf %x\\n $h +} + +verbose() { :; } +[ "${MVNW_VERBOSE-}" != true ] || verbose() { printf %s\\n "${1-}"; } + +die() { + printf %s\\n "$1" >&2 + exit 1 +} + +trim() { + # MWRAPPER-139: + # Trims trailing and leading whitespace, carriage returns, tabs, and linefeeds. + # Needed for removing poorly interpreted newline sequences when running in more + # exotic environments such as mingw bash on Windows. + printf "%s" "${1}" | tr -d '[:space:]' +} + +scriptDir="$(dirname "$0")" +scriptName="$(basename "$0")" + +# parse distributionUrl and optional distributionSha256Sum, requires .mvn/wrapper/maven-wrapper.properties +while IFS="=" read -r key value; do + case "${key-}" in + distributionUrl) distributionUrl=$(trim "${value-}") ;; + distributionSha256Sum) distributionSha256Sum=$(trim "${value-}") ;; + esac +done <"$scriptDir/.mvn/wrapper/maven-wrapper.properties" +[ -n "${distributionUrl-}" ] || die "cannot read distributionUrl property in $scriptDir/.mvn/wrapper/maven-wrapper.properties" + +case "${distributionUrl##*/}" in +maven-mvnd-*bin.*) + MVN_CMD=mvnd.sh _MVNW_REPO_PATTERN=/maven/mvnd/ + case "${PROCESSOR_ARCHITECTURE-}${PROCESSOR_ARCHITEW6432-}:$(uname -a)" in + *AMD64:CYGWIN* | *AMD64:MINGW*) distributionPlatform=windows-amd64 ;; + :Darwin*x86_64) distributionPlatform=darwin-amd64 ;; + :Darwin*arm64) distributionPlatform=darwin-aarch64 ;; + :Linux*x86_64*) distributionPlatform=linux-amd64 ;; + *) + echo "Cannot detect native platform for mvnd on $(uname)-$(uname -m), use pure java version" >&2 + distributionPlatform=linux-amd64 + ;; + esac + distributionUrl="${distributionUrl%-bin.*}-$distributionPlatform.zip" + ;; +maven-mvnd-*) MVN_CMD=mvnd.sh _MVNW_REPO_PATTERN=/maven/mvnd/ ;; +*) MVN_CMD="mvn${scriptName#mvnw}" _MVNW_REPO_PATTERN=/org/apache/maven/ ;; +esac + +# apply MVNW_REPOURL and calculate MAVEN_HOME +# maven home pattern: ~/.m2/wrapper/dists/{apache-maven-,maven-mvnd--}/ +[ -z "${MVNW_REPOURL-}" ] || distributionUrl="$MVNW_REPOURL$_MVNW_REPO_PATTERN${distributionUrl#*"$_MVNW_REPO_PATTERN"}" +distributionUrlName="${distributionUrl##*/}" +distributionUrlNameMain="${distributionUrlName%.*}" +distributionUrlNameMain="${distributionUrlNameMain%-bin}" +MAVEN_USER_HOME="${MAVEN_USER_HOME:-${HOME}/.m2}" +MAVEN_HOME="${MAVEN_USER_HOME}/wrapper/dists/${distributionUrlNameMain-}/$(hash_string "$distributionUrl")" + +exec_maven() { + unset MVNW_VERBOSE MVNW_USERNAME MVNW_PASSWORD MVNW_REPOURL || : + exec "$MAVEN_HOME/bin/$MVN_CMD" "$@" || die "cannot exec $MAVEN_HOME/bin/$MVN_CMD" +} + +if [ -d "$MAVEN_HOME" ]; then + verbose "found existing MAVEN_HOME at $MAVEN_HOME" + exec_maven "$@" +fi + +case "${distributionUrl-}" in +*?-bin.zip | *?maven-mvnd-?*-?*.zip) ;; +*) die "distributionUrl is not valid, must match *-bin.zip or maven-mvnd-*.zip, but found '${distributionUrl-}'" ;; +esac + +# prepare tmp dir +if TMP_DOWNLOAD_DIR="$(mktemp -d)" && [ -d "$TMP_DOWNLOAD_DIR" ]; then + clean() { rm -rf -- "$TMP_DOWNLOAD_DIR"; } + trap clean HUP INT TERM EXIT +else + die "cannot create temp dir" +fi + +mkdir -p -- "${MAVEN_HOME%/*}" + +# Download and Install Apache Maven +verbose "Couldn't find MAVEN_HOME, downloading and installing it ..." +verbose "Downloading from: $distributionUrl" +verbose "Downloading to: $TMP_DOWNLOAD_DIR/$distributionUrlName" + +# select .zip or .tar.gz +if ! command -v unzip >/dev/null; then + distributionUrl="${distributionUrl%.zip}.tar.gz" + distributionUrlName="${distributionUrl##*/}" +fi + +# verbose opt +__MVNW_QUIET_WGET=--quiet __MVNW_QUIET_CURL=--silent __MVNW_QUIET_UNZIP=-q __MVNW_QUIET_TAR='' +[ "${MVNW_VERBOSE-}" != true ] || __MVNW_QUIET_WGET='' __MVNW_QUIET_CURL='' __MVNW_QUIET_UNZIP='' __MVNW_QUIET_TAR=v + +# normalize http auth +case "${MVNW_PASSWORD:+has-password}" in +'') MVNW_USERNAME='' MVNW_PASSWORD='' ;; +has-password) [ -n "${MVNW_USERNAME-}" ] || MVNW_USERNAME='' MVNW_PASSWORD='' ;; +esac + +if [ -z "${MVNW_USERNAME-}" ] && command -v wget >/dev/null; then + verbose "Found wget ... using wget" + wget ${__MVNW_QUIET_WGET:+"$__MVNW_QUIET_WGET"} "$distributionUrl" -O "$TMP_DOWNLOAD_DIR/$distributionUrlName" || die "wget: Failed to fetch $distributionUrl" +elif [ -z "${MVNW_USERNAME-}" ] && command -v curl >/dev/null; then + verbose "Found curl ... using curl" + curl ${__MVNW_QUIET_CURL:+"$__MVNW_QUIET_CURL"} -f -L -o "$TMP_DOWNLOAD_DIR/$distributionUrlName" "$distributionUrl" || die "curl: Failed to fetch $distributionUrl" +elif set_java_home; then + verbose "Falling back to use Java to download" + javaSource="$TMP_DOWNLOAD_DIR/Downloader.java" + targetZip="$TMP_DOWNLOAD_DIR/$distributionUrlName" + cat >"$javaSource" <<-END + public class Downloader extends java.net.Authenticator + { + protected java.net.PasswordAuthentication getPasswordAuthentication() + { + return new java.net.PasswordAuthentication( System.getenv( "MVNW_USERNAME" ), System.getenv( "MVNW_PASSWORD" ).toCharArray() ); + } + public static void main( String[] args ) throws Exception + { + setDefault( new Downloader() ); + java.nio.file.Files.copy( java.net.URI.create( args[0] ).toURL().openStream(), java.nio.file.Paths.get( args[1] ).toAbsolutePath().normalize() ); + } + } + END + # For Cygwin/MinGW, switch paths to Windows format before running javac and java + verbose " - Compiling Downloader.java ..." + "$(native_path "$JAVACCMD")" "$(native_path "$javaSource")" || die "Failed to compile Downloader.java" + verbose " - Running Downloader.java ..." + "$(native_path "$JAVACMD")" -cp "$(native_path "$TMP_DOWNLOAD_DIR")" Downloader "$distributionUrl" "$(native_path "$targetZip")" +fi + +# If specified, validate the SHA-256 sum of the Maven distribution zip file +if [ -n "${distributionSha256Sum-}" ]; then + distributionSha256Result=false + if [ "$MVN_CMD" = mvnd.sh ]; then + echo "Checksum validation is not supported for maven-mvnd." >&2 + echo "Please disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties." >&2 + exit 1 + elif command -v sha256sum >/dev/null; then + if echo "$distributionSha256Sum $TMP_DOWNLOAD_DIR/$distributionUrlName" | sha256sum -c - >/dev/null 2>&1; then + distributionSha256Result=true + fi + elif command -v shasum >/dev/null; then + if echo "$distributionSha256Sum $TMP_DOWNLOAD_DIR/$distributionUrlName" | shasum -a 256 -c >/dev/null 2>&1; then + distributionSha256Result=true + fi + else + echo "Checksum validation was requested but neither 'sha256sum' or 'shasum' are available." >&2 + echo "Please install either command, or disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties." >&2 + exit 1 + fi + if [ $distributionSha256Result = false ]; then + echo "Error: Failed to validate Maven distribution SHA-256, your Maven distribution might be compromised." >&2 + echo "If you updated your Maven version, you need to update the specified distributionSha256Sum property." >&2 + exit 1 + fi +fi + +# unzip and move +if command -v unzip >/dev/null; then + unzip ${__MVNW_QUIET_UNZIP:+"$__MVNW_QUIET_UNZIP"} "$TMP_DOWNLOAD_DIR/$distributionUrlName" -d "$TMP_DOWNLOAD_DIR" || die "failed to unzip" +else + tar xzf${__MVNW_QUIET_TAR:+"$__MVNW_QUIET_TAR"} "$TMP_DOWNLOAD_DIR/$distributionUrlName" -C "$TMP_DOWNLOAD_DIR" || die "failed to untar" +fi + +# Find the actual extracted directory name (handles snapshots where filename != directory name) +actualDistributionDir="" + +# First try the expected directory name (for regular distributions) +if [ -d "$TMP_DOWNLOAD_DIR/$distributionUrlNameMain" ]; then + if [ -f "$TMP_DOWNLOAD_DIR/$distributionUrlNameMain/bin/$MVN_CMD" ]; then + actualDistributionDir="$distributionUrlNameMain" + fi +fi + +# If not found, search for any directory with the Maven executable (for snapshots) +if [ -z "$actualDistributionDir" ]; then + # enable globbing to iterate over items + set +f + for dir in "$TMP_DOWNLOAD_DIR"/*; do + if [ -d "$dir" ]; then + if [ -f "$dir/bin/$MVN_CMD" ]; then + actualDistributionDir="$(basename "$dir")" + break + fi + fi + done + set -f +fi + +if [ -z "$actualDistributionDir" ]; then + verbose "Contents of $TMP_DOWNLOAD_DIR:" + verbose "$(ls -la "$TMP_DOWNLOAD_DIR")" + die "Could not find Maven distribution directory in extracted archive" +fi + +verbose "Found extracted Maven distribution directory: $actualDistributionDir" +printf %s\\n "$distributionUrl" >"$TMP_DOWNLOAD_DIR/$actualDistributionDir/mvnw.url" +mv -- "$TMP_DOWNLOAD_DIR/$actualDistributionDir" "$MAVEN_HOME" || [ -d "$MAVEN_HOME" ] || die "fail to move MAVEN_HOME" + +clean || : +exec_maven "$@" diff --git a/pom.xml b/pom.xml new file mode 100644 index 00000000..05965106 --- /dev/null +++ b/pom.xml @@ -0,0 +1,113 @@ + + + 4.0.0 + + org.springframework.boot + spring-boot-starter-parent + 4.1.0 + + + com.example + TechnicalAssessment + 0.0.1-SNAPSHOT + + + + + + + + + + + + + + + + + 21 + + + + org.springframework.boot + spring-boot-starter-webmvc + + + org.springframework.boot + spring-boot-starter-validation + + + + org.springframework.boot + spring-boot-devtools + runtime + true + + + org.projectlombok + lombok + true + + + org.springframework.boot + spring-boot-starter-webmvc-test + test + + + + + + + org.springframework.boot + spring-boot-maven-plugin + + + + org.projectlombok + lombok + + + + + + org.apache.maven.plugins + maven-compiler-plugin + + + default-compile + compile + + compile + + + + + org.projectlombok + lombok + + + + + + default-testCompile + test-compile + + testCompile + + + + + org.projectlombok + lombok + + + + + + + + + + diff --git a/src/main/java/com/example/TechnicalAssessment/TechnicalAssessmentApplication.java b/src/main/java/com/example/TechnicalAssessment/TechnicalAssessmentApplication.java new file mode 100644 index 00000000..7ccb0d60 --- /dev/null +++ b/src/main/java/com/example/TechnicalAssessment/TechnicalAssessmentApplication.java @@ -0,0 +1,13 @@ +package com.example.TechnicalAssessment; + +import org.springframework.boot.SpringApplication; +import org.springframework.boot.autoconfigure.SpringBootApplication; + +@SpringBootApplication +public class TechnicalAssessmentApplication { + + public static void main(String[] args) { + SpringApplication.run(TechnicalAssessmentApplication.class, args); + } + +} diff --git a/src/main/java/com/example/TechnicalAssessment/client/ProductApiClient.java b/src/main/java/com/example/TechnicalAssessment/client/ProductApiClient.java new file mode 100644 index 00000000..5ca83858 --- /dev/null +++ b/src/main/java/com/example/TechnicalAssessment/client/ProductApiClient.java @@ -0,0 +1,72 @@ +package com.example.TechnicalAssessment.client; + +import com.example.TechnicalAssessment.exception.ExternalApiException; +import com.example.TechnicalAssessment.exception.ProductNotFoundException; +import com.example.TechnicalAssessment.model.ProductDetail; +import java.util.Arrays; +import java.util.List; +import org.springframework.beans.factory.annotation.Value; +import org.springframework.http.HttpStatusCode; +import org.springframework.stereotype.Component; +import org.springframework.web.client.ResourceAccessException; +import org.springframework.web.client.RestClient; + +@Component +public class ProductApiClient { + + private final RestClient restClient; + + public ProductApiClient( + RestClient.Builder restClientBuilder, + @Value("${external.api.base-url:http://localhost:3001}") String externalApiBaseUrl + ) { + this.restClient = restClientBuilder + .baseUrl(externalApiBaseUrl) + .build(); + } + + public List getSimilarProductIds(String productId) { + try { + String[] similarIds = restClient.get() + .uri("/product/{productId}/similarids", productId) + .retrieve() + .onStatus(HttpStatusCode::isError, (request, response) -> { + throw new ExternalApiException( + "Error retrieving similar product ids for product " + productId + ); + }) + .body(String[].class); + + return similarIds == null ? List.of() : Arrays.asList(similarIds); + } catch (ResourceAccessException exception) { + throw new ExternalApiException( + "External API is unavailable while retrieving similar product ids for product " + + productId, + exception + ); + } + } + + public ProductDetail getProductDetail(String productId) { + try { + return restClient.get() + .uri("/product/{productId}", productId) + .retrieve() + .onStatus( + statusCode -> statusCode.value() == 404, + (request, response) -> { + throw new ProductNotFoundException("Product not found: " + productId); + } + ) + .onStatus(HttpStatusCode::isError, (request, response) -> { + throw new ExternalApiException("Error retrieving product detail for product " + productId); + }) + .body(ProductDetail.class); + } catch (ResourceAccessException exception) { + throw new ExternalApiException( + "External API is unavailable while retrieving product detail for product " + productId, + exception + ); + } + } +} diff --git a/src/main/java/com/example/TechnicalAssessment/config/RestClientConfig.java b/src/main/java/com/example/TechnicalAssessment/config/RestClientConfig.java new file mode 100644 index 00000000..1ec22bc7 --- /dev/null +++ b/src/main/java/com/example/TechnicalAssessment/config/RestClientConfig.java @@ -0,0 +1,25 @@ +package com.example.TechnicalAssessment.config; + +import java.net.http.HttpClient; +import java.time.Duration; +import org.springframework.context.annotation.Bean; +import org.springframework.context.annotation.Configuration; +import org.springframework.http.client.JdkClientHttpRequestFactory; +import org.springframework.web.client.RestClient; + +@Configuration +public class RestClientConfig { + + @Bean + public RestClient.Builder restClientBuilder() { + HttpClient httpClient = HttpClient.newBuilder() + .connectTimeout(Duration.ofSeconds(2)) + .build(); + + JdkClientHttpRequestFactory requestFactory = new JdkClientHttpRequestFactory(httpClient); + requestFactory.setReadTimeout(Duration.ofSeconds(3)); + + return RestClient.builder() + .requestFactory(requestFactory); + } +} diff --git a/src/main/java/com/example/TechnicalAssessment/controller/ProductController.java b/src/main/java/com/example/TechnicalAssessment/controller/ProductController.java new file mode 100644 index 00000000..726b13f2 --- /dev/null +++ b/src/main/java/com/example/TechnicalAssessment/controller/ProductController.java @@ -0,0 +1,26 @@ +package com.example.TechnicalAssessment.controller; + +import com.example.TechnicalAssessment.model.ProductDetail; +import com.example.TechnicalAssessment.service.SimilarProductService; +import java.util.List; +import org.springframework.http.ResponseEntity; +import org.springframework.web.bind.annotation.GetMapping; +import org.springframework.web.bind.annotation.PathVariable; +import org.springframework.web.bind.annotation.RequestMapping; +import org.springframework.web.bind.annotation.RestController; + +@RestController +@RequestMapping("/product") +public class ProductController { + + private final SimilarProductService similarProductService; + + public ProductController(SimilarProductService similarProductService) { + this.similarProductService = similarProductService; + } + + @GetMapping("/{productId}/similar") + public ResponseEntity> getSimilarProducts(@PathVariable String productId) { + return ResponseEntity.ok(similarProductService.getSimilarProducts(productId)); + } +} diff --git a/src/main/java/com/example/TechnicalAssessment/exception/ExternalApiException.java b/src/main/java/com/example/TechnicalAssessment/exception/ExternalApiException.java new file mode 100644 index 00000000..2da94c89 --- /dev/null +++ b/src/main/java/com/example/TechnicalAssessment/exception/ExternalApiException.java @@ -0,0 +1,12 @@ +package com.example.TechnicalAssessment.exception; + +public class ExternalApiException extends RuntimeException { + + public ExternalApiException(String message) { + super(message); + } + + public ExternalApiException(String message, Throwable cause) { + super(message, cause); + } +} diff --git a/src/main/java/com/example/TechnicalAssessment/exception/GlobalExceptionHandler.java b/src/main/java/com/example/TechnicalAssessment/exception/GlobalExceptionHandler.java new file mode 100644 index 00000000..7ec8c08e --- /dev/null +++ b/src/main/java/com/example/TechnicalAssessment/exception/GlobalExceptionHandler.java @@ -0,0 +1,22 @@ +package com.example.TechnicalAssessment.exception; + +import java.util.Map; +import org.springframework.http.HttpStatus; +import org.springframework.http.ResponseEntity; +import org.springframework.web.bind.annotation.ExceptionHandler; +import org.springframework.web.bind.annotation.RestControllerAdvice; + +@RestControllerAdvice +public class GlobalExceptionHandler { + + @ExceptionHandler(ProductNotFoundException.class) + public ResponseEntity handleProductNotFound(ProductNotFoundException exception) { + return ResponseEntity.notFound().build(); + } + + @ExceptionHandler(ExternalApiException.class) + public ResponseEntity> handleExternalApiException(ExternalApiException exception) { + return ResponseEntity.status(HttpStatus.BAD_GATEWAY) + .body(Map.of("message", exception.getMessage())); + } +} diff --git a/src/main/java/com/example/TechnicalAssessment/exception/ProductNotFoundException.java b/src/main/java/com/example/TechnicalAssessment/exception/ProductNotFoundException.java new file mode 100644 index 00000000..1d6b0755 --- /dev/null +++ b/src/main/java/com/example/TechnicalAssessment/exception/ProductNotFoundException.java @@ -0,0 +1,8 @@ +package com.example.TechnicalAssessment.exception; + +public class ProductNotFoundException extends RuntimeException { + + public ProductNotFoundException(String message) { + super(message); + } +} diff --git a/src/main/java/com/example/TechnicalAssessment/model/ProductDetail.java b/src/main/java/com/example/TechnicalAssessment/model/ProductDetail.java new file mode 100644 index 00000000..b3dfcc78 --- /dev/null +++ b/src/main/java/com/example/TechnicalAssessment/model/ProductDetail.java @@ -0,0 +1,11 @@ +package com.example.TechnicalAssessment.model; + +import java.math.BigDecimal; + +public record ProductDetail( + String id, + String name, + BigDecimal price, + boolean availability +) { +} diff --git a/src/main/java/com/example/TechnicalAssessment/service/SimilarProductService.java b/src/main/java/com/example/TechnicalAssessment/service/SimilarProductService.java new file mode 100644 index 00000000..5555361b --- /dev/null +++ b/src/main/java/com/example/TechnicalAssessment/service/SimilarProductService.java @@ -0,0 +1,44 @@ +package com.example.TechnicalAssessment.service; + +import com.example.TechnicalAssessment.client.ProductApiClient; +import com.example.TechnicalAssessment.model.ProductDetail; +import java.util.List; +import java.util.concurrent.CompletableFuture; +import java.util.concurrent.Executor; +import org.springframework.stereotype.Service; + +@Service +public class SimilarProductService { + + private final ProductApiClient productApiClient; + private final Executor virtualThreadTaskExecutor; + + public SimilarProductService( + ProductApiClient productApiClient, + Executor virtualThreadTaskExecutor + ) { + this.productApiClient = productApiClient; + this.virtualThreadTaskExecutor = virtualThreadTaskExecutor; + } + + public List getSimilarProducts(String productId) { + List similarProductIds = productApiClient.getSimilarProductIds(productId); + + if (similarProductIds.isEmpty()) { + return List.of(); + } + + List> detailFutures = similarProductIds.stream() + .distinct() + .map(similarProductId -> CompletableFuture.supplyAsync( + () -> productApiClient.getProductDetail(similarProductId), + virtualThreadTaskExecutor + ).exceptionally(exception -> null)) + .toList(); + + return detailFutures.stream() + .map(CompletableFuture::join) + .filter(productDetail -> productDetail != null) + .toList(); + } +} diff --git a/src/main/resources/application.properties b/src/main/resources/application.properties new file mode 100644 index 00000000..2138d4b8 --- /dev/null +++ b/src/main/resources/application.properties @@ -0,0 +1,4 @@ +spring.application.name=TechnicalAssessment +server.port=5000 +external.api.base-url=http://localhost:3001 +spring.threads.virtual.enabled=true