Skip to content

Conversation

@bobbai00
Copy link
Contributor

@bobbai00 bobbai00 commented Mar 5, 2024

This PR introduces the GUI of environment and some fixes to previous dataset features. For the backend of environment, see #2434

Features

  • Environment Tab on the left panel
    2024-03-04 23 26 19

  • Auto Complete only from the files in datasets
    2024-03-04 23 27 32

Implementation Details

  1. The changes on the ScanSourceOperatorDesc

Previously, the source file is located by its absolute path and scanned into the workflow. Now, since all the files are within the dataset and managed by JGit, its physical file may not be directly available, current solution is to write the target file into a temporary file, which is identified by an absolute path generated by JVM. The file will be deleted when JVM quits.

  1. When workflow execute request is submitted, the webserver will also persist the environment eid to the WorkflowExecutions table.

@bobbai00 bobbai00 requested a review from aglinxinyuan March 5, 2024 07:38
@bobbai00 bobbai00 self-assigned this Mar 5, 2024
@bobbai00 bobbai00 force-pushed the jiadong-introduce-environment-gui branch from 1977090 to 0ba97cc Compare March 5, 2024 16:31
Copy link
Contributor

@aglinxinyuan aglinxinyuan left a comment

Choose a reason for hiding this comment

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

LGTM!

@bobbai00 bobbai00 merged commit f840b09 into master Mar 6, 2024
@bobbai00 bobbai00 deleted the jiadong-introduce-environment-gui branch March 6, 2024 00:55
@bobbai00 bobbai00 added gui ddl-change Changes to the TexeraDB DDL and removed ddl-change Changes to the TexeraDB DDL labels Mar 6, 2024
bobbai00 added a commit that referenced this pull request Mar 7, 2024
@Yicong-Huang Yicong-Huang added frontend Changes related to the frontend GUI and removed gui labels Jul 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend Changes related to the frontend GUI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants