All Products
Search
Document Center

:Code review

Last Updated:Dec 10, 2024

You can enable the forcible code review feature only for workspaces. After you enable the forcible code review feature for a workspace, the node code that is committed by developers in the workspace can be deployed only after the node code passes the code review. You can specify the nodes for which you want to initiate code review based on the priority of the baseline to which the nodes belong. For example, you can initiate code review for nodes that belong to a baseline that is assigned a higher priority. This way, the nodes can run as expected and do not block other nodes. This topic describes how to use the forcible code review feature.

Prerequisites

  • DataWorks Professional Edition or a more advanced edition is activated. For information about the differences in features of DataWorks Standard Edition, DataWorks Professional Edition, and DataWorks Enterprise Edition, see Differences among DataWorks editions. For information about how to activate DataWorks, see Purchase guide.

  • You are assigned the Workspace Administrator role. For more information, see Overview of users, roles, and permissions.

  • A node is created for data development. For more information, see Overview.

Background information

If multiple nodes are run in parallel in DataWorks, nodes that are assigned higher priorities can preferentially use resources. If the nodes that are assigned higher priorities fail to run due to invalid node code, the nodes occupy computing resources for an extended period of time. As a result, resources are wasted and other nodes may be blocked from running. To resolve this issue, we recommend that you add the nodes that are assigned higher priorities to a specific baseline and enable the forcible code review feature for the baseline. This way, a code reviewer can control the code quality of the nodes, and prevent node execution errors that occur due to invalid node code.

The following table describes the impacts of the forcible code review feature on the node commit and deployment processes.

Status of forcible code review

Description

Impact

Disabled

When you commit a node, DataWorks notifies the specified code reviewer of node changes.

Note

All members in the workspace, regardless of their roles, can be configured as the code reviewer.

The node commit and deployment processes are not affected. The node can be normally committed and deployed.

Enabled

When you commit a node, a qualified user must be specified to review the code of the node to control the code quality.

Note

If you delete a node, code review is not triggered when you commit the node.

The node commit and deployment processes are blocked and code review is performed on the node.

  • If the node code passes the code review, the node can be committed and deployed.

  • If the node code fails to pass the code review, the node cannot be committed or deployed. You must handle the issues in the code and proceed to the processes after the node code passes the code review.

Limits

  • Only users of DataWorks Professional Edition or a more advanced edition can use the forcible code review feature.

  • Only users who are assigned the Workspace Administrator role can determine whether to enable the forcible code review feature in a workspace.

  • You cannot initiate code review for some special types of nodes. For example, you cannot initiate code review for a combined node such as a do-while node or a for-each node, or a non-Python resource file.

Procedure

The following table describes the procedure for using the forcible code review feature.

Step

Execution role

Description

Enable the forcible code review feature and configure the parameters

Workspace administrator

Users that are assigned the Workspace Administrator role can enable the forcible code review feature, specify code reviewers, and specify the code review scope based on their business requirements. The forcible code review feature takes effect only after the feature is enabled.

Note

After the forcible code review feature is enabled, the node commit and deployment processes are blocked.

Initiate code review for nodes

Developer

Code review is triggered when a developer commits a node. After the developer specifies a code reviewer, the system generates a code review ticket.

Note

In a workspace in standard mode, a developer can select multiple nodes and initiate code review for the nodes at a time on the Deploy page. For more information, see Deploy nodes.

View and process code review tickets

  • Code review initiator

  • Code reviewer

On the Code Review page, a code review initiator can view the nodes for which they initiated code review, and code reviewers can view and process the nodes whose code they need to review.

Enable the forcible code review feature and configure the parameters

  1. Log on to the DataWorks console. In the top navigation bar, select the desired region. In the left-side navigation pane, choose Data Development and Governance > Data Development. On the page that appears, select the desired workspace from the drop-down list and click Go to Data Development.

  2. Go to the Security Settings and Others tab.

    1. In the lower-left corner of the DataStudio page, click the 设置 icon. The Personal Settings tab of the Settings page appears.

    2. Click the Security Settings and Others tab.

  3. Enable the forcible code review feature and configure the parameters.

    In the Code Review section of the Security Settings and Others tab, you can turn on Force to review code and configure the parameters: Code reviewers and Baseline scopes for code review.配置评审信息

    Parameter

    Description

    Force to review code

    Specifies whether to enable the forcible code review feature for specific nodes in the current workspace. If you turn on the switch, code review is triggered when you commit the nodes.

    • On: You must specify a code reviewer when you commit specific nodes in the current workspace. The node code can be deployed only after the code passes the code review. If the forcible code review feature is enabled, the node commit and deployment processes are blocked.

      Note

      You can configure the Code reviewers parameter and Baseline scopes for code review parameter only after you turn on Force to review code.

    • Off: The code of the nodes that you commit in the current workspace can be deployed without the need for code review. You can specify a code reviewer when you select the nodes that you want to commit. After you commit the nodes, the specified code reviewer is only notified of changes to the nodes and the node commit and deployment processes are not blocked.

    Code reviewers

    The users who are available for code review when you commit the nodes.

    • Any Developer Role: If you select this option, all users who are assigned the Develop role in the current workspace are available for code review when you commit nodes. When you commit nodes, you must select a specific user to review the node code.

    • Specify development role users: If you select this option, you must specify a specific user as the code reviewer in this step. By default, the specified user is the code reviewer for nodes in the current workspace.

    Note
    • You can select only a user who is assigned the Develop role in the current workspace as a code reviewer.

    • DataWorks allows you to configure one or more code reviewers. If you select Any Developer Role in this step and configure multiple code reviewers when you commit nodes, the node code passes the code review after any of the code reviewers approves the code. If you select Specify development role users and configure multiple code reviewers in this step, the node code passes the code review only after all code reviewers approve the code.

    Baseline scopes for code review

    The scope of nodes on which you want to perform code review when you commit the nodes.

    You can determine the nodes on which you want to perform code review based on the priority of the baseline to which the nodes belong.

    • If you set this parameter to Non-baseline tasks, code review is performed on newly created nodes in the current workspace.

    • If you set this parameter to baseline tasks of one or more specific levels, code review is performed on the nodes that belong to the baselines of specific priorities in the current workspace.

    • A larger value of the baseline task level indicates a higher priority. Nodes in a baseline have a higher priority than the nodes that are not in a baseline.

    For more information about how to control the priorities of nodes by using a baseline, see Overview.

Initiate code review for nodes

DataWorks allows developers to specify one or more code reviewers when the developers commit nodes. The following table describes the code review process for workspaces in different modes.

Note
  • If you initiate code review for a node again when the previous code review is being performed for the code, the previous code review result becomes invalid.

  • If you delete a node, code review is not triggered when you commit the node.

Workspace mode

Code review process

Code review entry

Standard mode

If the forcible code review feature is enabled, the node commit and deployment processes are blocked until the node code passes the code review. After the node code passes the code review, the node is committed and you can deploy the node on the Deploy page.

  • Initiate code review for a single node

    • You can initiate code review for a single node in the dialog box that appears when you commit the node.

    • You can click the 代码评审 icon in the top toolbar on the node configuration tab when you commit a single node to initiate code review for the node.

  • Initiate code review for multiple nodes at a time

    • You can initiate code review for multiple nodes at a time in the dialog box that appears when you commit the workflow to which the nodes belong.

    • Go to the Deploy page and click Create Package in the left-side navigation pane. On the Create Deploy Task page, initiate code review for multiple nodes at a time.

      Note

      For more information about how to go to the Create Deploy Task page, see Deploy nodes.

Basic mode

If the forcible code review feature is enabled and you click Only Initiate Code Review in the dialog box that appears when you commit the node, you must click Submit again to commit the node to the production environment after the node code passes the code review.

  • Initiate code review for a single node

    You can initiate code review for a single node in the dialog box that appears when you commit the node.

  • Initiate code review for multiple nodes at a time

    You can initiate code review for multiple nodes at a time in the dialog box that appears when you commit the workflow to which the nodes belong.

View and process code review tickets

  1. Go to the Code Review page.

    You can go to the Code Review page by using one of the following methods:

    • In the upper-left corner of the DataStudio page, click the 全部 icon and choose All Products > Data Development And Task Operation > Code Review.

    • On the configuration tab of a node, click the 代码评审 icon in the top toolbar.

  2. View and process code review tickets.

    On the Code Review page, you can view the nodes for which you initiated code review and the nodes whose code needs to be reviewed by you. You can review the code of the nodes based on your business scenario.查看评审单

    Area

    Description

    1

    You can perform the following operations in this area:

    • A code reviewer can view and process the nodes whose code they need to review on the Review by Me page.

    • A code review initiator can view the nodes for which they initiated code review on the Submitted by Me page.

    2

    In this area, you can search for nodes that meet the specified filter conditions such as the code review status, node name, node type, code review initiator, and node commit time.

    3

    In this area, you can view the basic information such as the version and code review status of a node. You can also view the details of a node and process the node.

    You can perform the following operations:

    • Version comparison: allows you to learn node changes based on a comparison of node code between different versions.

    • Processing of code review tickets:

      • Write a comment: allows you to add a comment on the node version.

      • Pass: triggers the code review checker to check the code of a node. After the node code passes the code review, the code review initiator can deploy the node on the Deploy page.

        Note

        For more information about how to deploy a node, see Deploy nodes.

      • Do not pass: A node is processed based on whether Force to review code is turned on in the current workspace. If the switch is turned on, the node commit and deployment processes are blocked. If the switch is turned off, the node can be committed and deployed.

      • Abandoned: The current code review process is abandoned. After the code review process is abandoned, the node of the current version can continue to be committed and deployed.

      • Reopen: The abandoned code review process is reopened.