fix(TCOMP-2710): improve workflow and exception throwing/handling

component-api/src/main/java/org/talend/sdk/component/api/exception/DiscoverSchemaException.java

@@ -16,6 +16,7 @@
 package org.talend.sdk.component.api.exception;
 
 import javax.json.bind.annotation.JsonbCreator;
+import javax.json.bind.annotation.JsonbPropertyOrder;
 
 import lombok.Data;
 
@@ -28,13 +29,34 @@
  * See me TCOMP-2342 for more details.
  */
 @Data
+@JsonbPropertyOrder({ "localizedMessage", "message", "stackTrace", "suppressed", "possibleHandleErrorWith" })
 public class DiscoverSchemaException extends RuntimeException {
 
     public enum HandleErrorWith {
+        /**
+         * default case
+         */
         EXCEPTION,
+        /**
+         * unhandled.
+         */
         SILENT,
+        /**
+         * unhandled.
+         */
         RETRY,
-        EXECUTE_MOCK_JOB;
+        /**
+         * Potentially execute a mock job in studio.
+         * Will ask user's validation before executing.
+         * When specifying this option, developer should be sure that no side effect can be generated by connector.
+         */
+        EXECUTE_MOCK_JOB,
+        /**
+         * Will execute a lifecycle through framework using only configuration.
+         * This won't query for any user input.
+         * When specifying this option, developer should be sure that no side effect can be generated by connector.
+         */
+        EXECUTE_LIFECYCLE;
     }
 
     private HandleErrorWith possibleHandleErrorWith = HandleErrorWith.EXCEPTION;

component-studio/component-runtime-di/src/main/java/org/talend/sdk/component/runtime/di/schema/TaCoKitGuessSchema.java

@@ -19,7 +19,7 @@
 import static java.util.Optional.ofNullable;
 import static java.util.stream.Collectors.toList;
 import static java.util.stream.Collectors.toMap;
-import static org.talend.sdk.component.api.exception.DiscoverSchemaException.HandleErrorWith.EXECUTE_MOCK_JOB;
+import static org.talend.sdk.component.api.exception.DiscoverSchemaException.HandleErrorWith.EXECUTE_LIFECYCLE;
 import static org.talend.sdk.component.api.record.SchemaProperty.IS_KEY;
 import static org.talend.sdk.component.api.record.SchemaProperty.PATTERN;
 import static org.talend.sdk.component.api.record.SchemaProperty.SCALE;
@@ -202,13 +202,10 @@
         try {
             executeDiscoverSchemaExtendedAction(incomingSchema, outgoingBranch);
         } catch (Exception e) {
-            // Case when a processor is the start of a studio job
-            if (isStartOfJob) {
+            final DiscoverSchemaException dse = transformException(e);
+            // When a processor is the start of a studio job and dev explicitly set the handleError to Lifecycle exec
+            if (isStartOfJob && EXECUTE_LIFECYCLE.equals(dse.getPossibleHandleErrorWith())) {
                 try {
-                    final DiscoverSchemaException dse = transformException(e);
-                    if (EXECUTE_MOCK_JOB.equals(dse.getPossibleHandleErrorWith())) {
-                        throw handleException(dse);
-                    }
                     guessOutputComponentSchemaThroughResult();
                 } catch (Exception er) {
                     throw handleException(e);