Till Westmann has submitted this change and it was merged. Change subject: remove more useless curly braces ......................................................................
remove more useless curly braces Change-Id: I581d268b12017fcf17b5b29f7183e07deae6e931 Reviewed-on: https://asterix-gerrit.ics.uci.edu/1363 Reviewed-by: abdullah alamoudi <[email protected]> Sonar-Qube: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> --- M asterixdb/asterix-om/src/main/java/org/apache/asterix/formats/nontagged/AqlNormalizedKeyComputerFactoryProvider.java 1 file changed, 14 insertions(+), 28 deletions(-) Approvals: abdullah alamoudi: Looks good to me, approved Jenkins: Verified; No violations found diff --git a/asterixdb/asterix-om/src/main/java/org/apache/asterix/formats/nontagged/AqlNormalizedKeyComputerFactoryProvider.java b/asterixdb/asterix-om/src/main/java/org/apache/asterix/formats/nontagged/AqlNormalizedKeyComputerFactoryProvider.java index 8e424ec..a819b67 100644 --- a/asterixdb/asterix-om/src/main/java/org/apache/asterix/formats/nontagged/AqlNormalizedKeyComputerFactoryProvider.java +++ b/asterixdb/asterix-om/src/main/java/org/apache/asterix/formats/nontagged/AqlNormalizedKeyComputerFactoryProvider.java @@ -45,58 +45,44 @@ case DATE: case TIME: case YEARMONTHDURATION: - case INT32: { + case INT32: return new AWrappedAscNormalizedKeyComputerFactory(new IntegerNormalizedKeyComputerFactory()); - } case DATETIME: case DAYTIMEDURATION: - case INT64: { + case INT64: return new AWrappedAscNormalizedKeyComputerFactory(new Integer64NormalizedKeyComputerFactory()); - } - case FLOAT: { + case FLOAT: return new AWrappedAscNormalizedKeyComputerFactory(new FloatNormalizedKeyComputerFactory()); - } - case DOUBLE: { + case DOUBLE: return new AWrappedAscNormalizedKeyComputerFactory(new DoubleNormalizedKeyComputerFactory()); - } - case STRING: { + case STRING: return new AWrappedAscNormalizedKeyComputerFactory(new UTF8StringNormalizedKeyComputerFactory()); - } - case BINARY: { + case BINARY: return new AWrappedAscNormalizedKeyComputerFactory(new ByteArrayNormalizedKeyComputerFactory()); - } - default: { + default: return null; - } } } else { switch (aqlType.getTypeTag()) { case DATE: case TIME: case YEARMONTHDURATION: - case INT32: { + case INT32: return new AWrappedDescNormalizedKeyComputerFactory(new IntegerNormalizedKeyComputerFactory()); - } case DATETIME: case DAYTIMEDURATION: - case INT64: { + case INT64: return new AWrappedDescNormalizedKeyComputerFactory(new Integer64NormalizedKeyComputerFactory()); - } - case FLOAT: { + case FLOAT: return new AWrappedDescNormalizedKeyComputerFactory(new FloatNormalizedKeyComputerFactory()); - } - case DOUBLE: { + case DOUBLE: return new AWrappedDescNormalizedKeyComputerFactory(new DoubleNormalizedKeyComputerFactory()); - } - case STRING: { + case STRING: return new AWrappedDescNormalizedKeyComputerFactory(new UTF8StringNormalizedKeyComputerFactory()); - } - case BINARY: { + case BINARY: return new AWrappedDescNormalizedKeyComputerFactory(new ByteArrayNormalizedKeyComputerFactory()); - } - default: { + default: return null; - } } } } -- To view, visit https://asterix-gerrit.ics.uci.edu/1363 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I581d268b12017fcf17b5b29f7183e07deae6e931 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Till Westmann <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]>
