Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactoring for display column lineage JSON #2301

Conversation

tito12
Copy link
Contributor

@tito12 tito12 commented Dec 8, 2022

Problem

Loaded data could be heavy as request

Solution

Load after click on tab

Checklist

  • You've signed-off your work
  • Your changes are accompanied by tests (if relevant)
  • Your change contains a small diff and is self-contained
  • You've updated any relevant documentation (if relevant)
  • You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
  • You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)
  • You've included a header in any source code files (if relevant)

Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>
Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>
@boring-cyborg boring-cyborg bot added the web label Dec 8, 2022
@wslulciuc
Copy link
Member

@tito12, is this PR a duplication of #2293?

@codecov
Copy link

codecov bot commented Dec 8, 2022

Codecov Report

Merging #2301 (e97659e) into web/display-column-lineage-of-dataset (c003400) will not change coverage.
The diff coverage is n/a.

@@                           Coverage Diff                            @@
##             web/display-column-lineage-of-dataset    #2301   +/-   ##
========================================================================
  Coverage                                    76.97%   76.97%           
  Complexity                                    1163     1163           
========================================================================
  Files                                          222      222           
  Lines                                         5298     5298           
  Branches                                       423      423           
========================================================================
  Hits                                          4078     4078           
  Misses                                         747      747           
  Partials                                       473      473           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tito12 tito12 changed the title Web/display column lineage of dataset Refactoring for display column lineage JSON Dec 8, 2022
@tito12
Copy link
Contributor Author

tito12 commented Dec 8, 2022

@tito12, is this PR a duplication of #2293?

it was wrong title

@pawel-big-lebowski pawel-big-lebowski merged commit 51fae37 into MarquezProject:web/display-column-lineage-of-dataset Dec 8, 2022
pawel-big-lebowski pushed a commit that referenced this pull request Dec 12, 2022
* refactoring for display column lineage JSON

Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>

* request api after click

Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>

Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>
Co-authored-by: tito1212 <vladyslav.sedenko@getindata.com>
tito12 added a commit that referenced this pull request Dec 27, 2022
* display column lineage JSON

Signed-off-by: Pawel Leszczynski <leszczynski.pawel@gmail.com>

* refactoring for display column lineage JSON (#2294)

Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>

Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>
Co-authored-by: tito1212 <vladyslav.sedenko@getindata.com>

* Refactoring for display column lineage JSON  (#2301)

* refactoring for display column lineage JSON

Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>

* request api after click

Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>

Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>
Co-authored-by: tito1212 <vladyslav.sedenko@getindata.com>

* call backend on tab switch

Signed-off-by: Pawel Leszczynski <leszczynski.pawel@gmail.com>

* changes after review. refactoring. fix type and name of file

Signed-off-by: tito12 <vladyslav.sedenko@gmail.com>

* Changes after review. Separate fetching data in child component

Signed-off-by: tito12 <vladyslav.sedenko@gmail.com>

* eslint fix changes

Signed-off-by: tito12 <vladyslav.sedenko@gmail.com>

Signed-off-by: Pawel Leszczynski <leszczynski.pawel@gmail.com>
Signed-off-by: tito1212 <vladyslav.sedenko@getindata.com>
Signed-off-by: tito12 <vladyslav.sedenko@gmail.com>
Co-authored-by: tito12 <vladyslav.sedenko@gmail.com>
Co-authored-by: tito1212 <vladyslav.sedenko@getindata.com>
Co-authored-by: Pawel Leszczynski <pawelleszczynski@Pawels-MacBook-Pro.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants