From 483ca73ed8953d8e843c8150a395b5a4f800bb26 Mon Sep 17 00:00:00 2001 From: zhilingc Date: Thu, 6 Feb 2020 22:52:57 +0700 Subject: [PATCH] Fix typo in split string length check --- sdk/java/src/main/java/com/gojek/feast/RequestUtil.java | 2 +- sdk/java/src/test/java/com/gojek/feast/RequestUtilTest.java | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/sdk/java/src/main/java/com/gojek/feast/RequestUtil.java b/sdk/java/src/main/java/com/gojek/feast/RequestUtil.java index 075c570c4e9..874196e92bd 100644 --- a/sdk/java/src/main/java/com/gojek/feast/RequestUtil.java +++ b/sdk/java/src/main/java/com/gojek/feast/RequestUtil.java @@ -61,7 +61,7 @@ public static List createFeatureRefs( "Feature id '%s' contains invalid version. Expected format: /:.", featureRefString)); } - } else if (projectSplit.length == 1) { + } else if (featureSplit.length == 1) { name = featureSplit[0]; } else { throw new IllegalArgumentException( diff --git a/sdk/java/src/test/java/com/gojek/feast/RequestUtilTest.java b/sdk/java/src/test/java/com/gojek/feast/RequestUtilTest.java index 1c58e9435c6..3b9429ad8f6 100644 --- a/sdk/java/src/test/java/com/gojek/feast/RequestUtilTest.java +++ b/sdk/java/src/test/java/com/gojek/feast/RequestUtilTest.java @@ -60,7 +60,7 @@ private static Stream provideValidFeatureIds() { Arrays.asList( "driver_project/driver_id:1", "driver_project/driver_name:1", - "booking_project/driver_name:1"), + "booking_project/driver_name"), Arrays.asList( FeatureReference.newBuilder() .setProject("driver_project") @@ -74,7 +74,6 @@ private static Stream provideValidFeatureIds() { .build(), FeatureReference.newBuilder() .setProject("booking_project") - .setVersion(1) .setName("driver_name") .build()))); }