Skip to content

dag add close buttion#3195

Merged
khadgarmage merged 34 commits intoapache:devfrom
BoYiZhang:dev
Jul 15, 2020
Merged

dag add close buttion#3195
khadgarmage merged 34 commits intoapache:devfrom
BoYiZhang:dev

Conversation

@BoYiZhang
Copy link
Copy Markdown
Contributor

dag add close buttion

#3190

@davidzollo davidzollo linked an issue Jul 13, 2020 that may be closed by this pull request
@davidzollo
Copy link
Copy Markdown
Contributor

@break60 please review , thx

@break60
Copy link
Copy Markdown
Contributor

break60 commented Jul 13, 2020

Re-update

@BoYiZhang
Copy link
Copy Markdown
Contributor Author

BoYiZhang commented Jul 13, 2020

e2e test error .

Please contact the person concerned for repair

thx

Run actions/upload-artifact@v1
##[error]Path does not exist /var/lib/docker/volumes/docker-swarm_dolphinscheduler-logs/_data
##[error]Exit code 1 returned from process: file name '/home/runner/runners/2.267.1/bin/Runner.PluginHost', arguments 'action "GitHub.Runner.Plugins.Artifact.PublishArtifact, Runner.Plugins"'.

@xingchun-chen
Copy link
Copy Markdown
Contributor

Since the close button is added to the dag page, the xpath path of the save button changes. You need to change the xpath path of CLICK_SAVE_WORKFLOW_BUTTON of org.apache.dolphinscheduler.locator.project.CreateWorkflowLocator
image

xingchun-chen and others added 3 commits July 13, 2020 16:35
in addition

update '- XX: largepagesizeinbytes = 128M' to '- XX: largepagesizeinbytes = 10M'
Copy link
Copy Markdown
Contributor

@xingchun-chen xingchun-chen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Copy link
Copy Markdown
Member

@gabry-lab gabry-lab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@qiaozhanwei
Copy link
Copy Markdown
Contributor

SonarCloud Code Analysis is not running . please retry commit. Thx

@BoYiZhang
Copy link
Copy Markdown
Contributor Author

SonarCloud Code Analysis is not running . please retry commit. Thx

It has been re submitted twice, but sonarcloud code analysis is not running

@break60
Copy link
Copy Markdown
Contributor

break60 commented Jul 14, 2020

+1

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@khadgarmage khadgarmage merged commit c453a5b into apache:dev Jul 15, 2020
tswstarplanet pushed a commit to tswstarplanet/incubator-dolphinscheduler that referenced this pull request Jul 15, 2020
* fix bug

Delete invalid field: executorcores

Modify verification prompt

* fix bug

Delete invalid field: executorcores

Modify verification prompt

* fix bug

Delete invalid field: executorcores

Modify verification prompt

* dag  add close button

* reset last version

* reset last version

* dag add close buttion

dag add close buttion

* update  CLICK_SAVE_WORKFLOW_BUTTON  xpath

* updae CLICK_SAVE_WORKFLOW_BUTTON xpath

* updae CLICK_SAVE_WORKFLOW_BUTTON xpath

* updae CLICK_SAVE_WORKFLOW_BUTTON xpath

* Update CreateWorkflowLocator.java

modify submit workflow button

* Update CreateWorkflowLocator.java

* Update CreateWorkflowLocator.java

modify CLICK_ADD_BUTTON

* Update CreateWorkflowLocator.java

delete print

* Update CreateWorkflowLocator.java

1

* Update CreateWorkflowLocator.java

1

* Setting '-XX:+DisableExplicitGC ' causes netty memory leaks

in addition

update '- XX: largepagesizeinbytes = 128M' to '- XX: largepagesizeinbytes = 10M'

* Update dag.vue

* Update dag.vue

* Update dag.vue

* Update CreateWorkflowLocator.java

* Revert "Setting '-XX:+DisableExplicitGC ' causes netty memory leaks"

This reverts commit 3a2cba7

Co-authored-by: dailidong <dailidong66@gmail.com>
Co-authored-by: xingchun-chen <55787491+xingchun-chen@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] DAG need to add a close button ?

7 participants