Repository: appsmithorg/appsmith
File: app/server/appsmith-server/src/main/java/com/appsmith/server/solutions/ce/ActionExecutionSolutionCEImpl.java (Java)

Code before:
import java.util.concurrent.TimeoutException;
import java.util.concurrent.atomic.AtomicLong;
import java.util.concurrent.atomic.AtomicReference;
import java.util.function.Function;
import java.util.regex.Matcher;
import java.util.regex.Pattern;
import java.util.stream.Collectors;

import static com.appsmith.external.constants.CommonFieldName.REDACTED_DATA;
import static com.appsmith.external.constants.spans.ActionSpan.ACTION_EXECUTION_CACHED_DATASOURCE;
import static com.appsmith.external.constants.spans.ActionSpan.ACTION_EXECUTION_DATASOURCE_CONTEXT;
import static com.appsmith.external.constants.spans.ActionSpan.ACTION_EXECUTION_EDITOR_CONFIG;
import static com.appsmith.external.constants.spans.ActionSpan.ACTION_EXECUTION_REQUEST_PARSING;
import static com.appsmith.external.constants.spans.ActionSpan.ACTION_EXECUTION_SERVER_EXECUTION;
import static com.appsmith.external.helpers.DataTypeStringUtils.getDisplayDataTypes;
import static com.appsmith.server.constants.ce.FieldNameCE.NONE;
import static com.appsmith.server.helpers.WidgetSuggestionHelper.getSuggestedWidgets;
import static java.lang.Boolean.FALSE;
import static java.lang.Boolean.TRUE;

@Slf4j
public class ActionExecutionSolutionCEImpl implements ActionExecutionSolutionCE {

    private static final String GET_ENVIRONMENT_ID = "getEnvironmentId";
    private static final String POPULATED_EXECUTE_ACTION_DTO_MONO = "populatedExecuteActionDTOMono";
    private static final String POPULATE_AND_EXECUTE_ACTION = "populateAndExecuteAction";
    private static final String GET_VALID_ACTION_FOR_EXECUTION = "getValidActionForExecution";
    private static final String GET_CACHED_PLUGIN_FOR_ACTION_EXECUTION = "getCachedPluginForActionExecution";
    private static final String GET_PLUGIN_EXECUTOR = "getPluginExecutor";
    private static final String GET_ACTION_EXECUTION_RESULT = "getActionExecutionResult";
    private static final String EXECUTE_ACTION = "executeAction";
    private final NewActionService newActionService;
    private final ActionPermission actionPermission;
    private final ObservationRegistry observationRegistry;
    private final ObjectMapper objectMapper;
    private final DatasourceService datasourceService;
    private final PluginService pluginService;
    private final DatasourceContextService datasourceContextService;
    private final PluginExecutorHelper pluginExecutorHelper;
    private final NewPageService newPageService;
    private final ApplicationService applicationService;
    private final SessionUserService sessionUserService;
    private final AuthenticationValidator authenticationValidator;
    private final DatasourcePermission datasourcePermission;
    private final AnalyticsService analyticsService;
    private final DatasourceStorageService datasourceStorageService;
    private final EnvironmentPermission environmentPermission;
    private final ConfigService configService;
    private final OrganizationService organizationService;
    private final ActionExecutionSolutionHelper actionExecutionSolutionHelper;
    private final CommonConfig commonConfig;

Review comment (suggestion):
@vsvamsi1 there is a class already with some span names `ActionSpanCE` do you want to add these there? or any reason you want to keep it here itself? it shouldn't be in 2 places 

Code after:
import java.util.concurrent.TimeoutException;
import java.util.concurrent.atomic.AtomicLong;
import java.util.concurrent.atomic.AtomicReference;
import java.util.function.Function;
import java.util.regex.Matcher;
import java.util.regex.Pattern;
import java.util.stream.Collectors;

import static com.appsmith.external.constants.CommonFieldName.REDACTED_DATA;
import static com.appsmith.external.constants.spans.ActionSpan.ACTION_EXECUTION_CACHED_DATASOURCE;
import static com.appsmith.external.constants.spans.ActionSpan.ACTION_EXECUTION_DATASOURCE_CONTEXT;
import static com.appsmith.external.constants.spans.ActionSpan.ACTION_EXECUTION_EDITOR_CONFIG;
import static com.appsmith.external.constants.spans.ActionSpan.ACTION_EXECUTION_REQUEST_PARSING;
import static com.appsmith.external.constants.spans.ActionSpan.ACTION_EXECUTION_SERVER_EXECUTION;
import static com.appsmith.external.constants.spans.ce.ActionSpanCE.*;
import static com.appsmith.external.helpers.DataTypeStringUtils.getDisplayDataTypes;
import static com.appsmith.server.constants.ce.FieldNameCE.NONE;
import static com.appsmith.server.helpers.WidgetSuggestionHelper.getSuggestedWidgets;
import static java.lang.Boolean.FALSE;
import static java.lang.Boolean.TRUE;

@Slf4j
public class ActionExecutionSolutionCEImpl implements ActionExecutionSolutionCE {

    private final NewActionService newActionService;
    private final ActionPermission actionPermission;
    private final ObservationRegistry observationRegistry;
    private final ObjectMapper objectMapper;
    private final DatasourceService datasourceService;
    private final PluginService pluginService;
    private final DatasourceContextService datasourceContextService;
    private final PluginExecutorHelper pluginExecutorHelper;
    private final NewPageService newPageService;
    private final ApplicationService applicationService;
    private final SessionUserService sessionUserService;
    private final AuthenticationValidator authenticationValidator;
    private final DatasourcePermission datasourcePermission;
    private final AnalyticsService analyticsService;
    private final DatasourceStorageService datasourceStorageService;
    private final EnvironmentPermission environmentPermission;
    private final ConfigService configService;
    private final OrganizationService organizationService;
    private final ActionExecutionSolutionHelper actionExecutionSolutionHelper;
    private final CommonConfig commonConfig;
    private final FeatureFlagService featureFlagService;

    static final String PARAM_KEY_REGEX = "^k\\d+$";
    static final String BLOB_KEY_REGEX =
            "^blob:[0-9a-fA-F]{8}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{4}\\b-[0-9a-fA-F]{12}$";
    static final String EXECUTE_ACTION_DTO = "executeActionDTO";
    static final String PARAMETER_MAP = "parameterMap";