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

fix(explore): revert dnd column dependency array change to fix infinite rerenders #16115

Merged
merged 3 commits into from Aug 6, 2021

Conversation

kgabryje
Copy link
Member

@kgabryje kgabryje commented Aug 6, 2021

SUMMARY

Revert change in dependency array in DndColumnSelect introduced in #16073 - it causes infinite rerenders and page freeze when interacting with the component.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

image

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

Copy link
Member

@michael-s-molina michael-s-molina left a comment

Choose a reason for hiding this comment

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

Approving because it broke the dnd.

We should address the comments in the original PR in a follow-up PR.

Screen Shot 2021-08-06 at 10 48 16 AM

@codecov
Copy link

codecov bot commented Aug 6, 2021

Codecov Report

Merging #16115 (189d1df) into master (b80f018) will increase coverage by 0.00%.
The diff coverage is n/a.

❗ Current head 189d1df differs from pull request most recent head f825c5f. Consider uploading reports for the commit f825c5f to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master   #16115   +/-   ##
=======================================
  Coverage   76.84%   76.85%           
=======================================
  Files         995      995           
  Lines       52886    52886           
  Branches     6721     6721           
=======================================
+ Hits        40640    40643    +3     
+ Misses      12021    12018    -3     
  Partials      225      225           
Flag Coverage Δ
javascript 71.23% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ontrols/DndColumnSelectControl/DndColumnSelect.tsx 46.42% <ø> (ø)
...ols/DndColumnSelectControl/utils/optionSelector.ts 46.15% <0.00%> (+11.53%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b80f018...f825c5f. Read the comment docs.

@kgabryje kgabryje merged commit 772da8d into apache:master Aug 6, 2021
@junlincc junlincc added preset:2021.31 #bug:blocking! Blocking issues with high priority and removed #bug:blocking! Blocking issues with high priority preset:2021.31 labels Aug 6, 2021
@rosemarie-chiu
Copy link
Contributor

🏷 2021.31

@villebro villebro added the v1.3 label Aug 10, 2021
villebro pushed a commit that referenced this pull request Aug 10, 2021
…te rerenders (#16115)

* fix(explore): revert dnd column dependency array change to fix infinite rerenders

* Remove console.log

* Remove comment

(cherry picked from commit 772da8d)
stevenuray pushed a commit to preset-io/superset that referenced this pull request Aug 11, 2021
…te rerenders (apache#16115)

* fix(explore): revert dnd column dependency array change to fix infinite rerenders

* Remove console.log

* Remove comment

(cherry picked from commit 772da8d)
opus-42 pushed a commit to opus-42/incubator-superset that referenced this pull request Nov 14, 2021
…te rerenders (apache#16115)

* fix(explore): revert dnd column dependency array change to fix infinite rerenders

* Remove console.log

* Remove comment
cccs-RyanS pushed a commit to CybercentreCanada/superset that referenced this pull request Dec 17, 2021
…te rerenders (apache#16115)

* fix(explore): revert dnd column dependency array change to fix infinite rerenders

* Remove console.log

* Remove comment

(cherry picked from commit 772da8d)
QAlexBall pushed a commit to QAlexBall/superset that referenced this pull request Dec 28, 2021
…te rerenders (apache#16115)

* fix(explore): revert dnd column dependency array change to fix infinite rerenders

* Remove console.log

* Remove comment
QAlexBall pushed a commit to QAlexBall/superset that referenced this pull request Dec 29, 2021
…te rerenders (apache#16115)

* fix(explore): revert dnd column dependency array change to fix infinite rerenders

* Remove console.log

* Remove comment

(cherry picked from commit 772da8d)
cccs-rc pushed a commit to CybercentreCanada/superset that referenced this pull request Mar 6, 2024
…te rerenders (apache#16115)

* fix(explore): revert dnd column dependency array change to fix infinite rerenders

* Remove console.log

* Remove comment

(cherry picked from commit 7408d55)
@mistercrunch mistercrunch added 🍒 1.3.0 🍒 1.3.1 🍒 1.3.2 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.4.0 labels Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels preset:2021.31 size/XS v1.3 🍒 1.3.0 🍒 1.3.1 🍒 1.3.2 🚢 1.4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants