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

disable autoOptimizeShuffle #795

Merged
merged 2 commits into from
Mar 6, 2023

Conversation

GeekSheikh
Copy link
Contributor

closes #794

@GeekSheikh GeekSheikh marked this pull request as ready for review March 2, 2023 21:51
@GeekSheikh GeekSheikh linked an issue Mar 2, 2023 that may be closed by this pull request
Copy link
Contributor

@brij-raghuwanshi-db brij-raghuwanshi-db left a comment

Choose a reason for hiding this comment

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

Based on the discussion and scenario, it makes sense.

@sonarcloud
Copy link

sonarcloud bot commented Mar 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@GeekSheikh GeekSheikh merged commit bf74d45 into 0712_release Mar 6, 2023
@GeekSheikh GeekSheikh deleted the 794_disable_autoOptimizeShuffle branch March 6, 2023 12:55
GeekSheikh added a commit that referenced this pull request Mar 6, 2023
* initial commit

* 788 bug aws single tenant multiworkspace deployment noneget (#798)

* Change Validate for MWS logic

* removed debug print

---------

Co-authored-by: sourav.banerjee <sourav.banerjee@databricks.com>

* disable autoOptimizeShuffle (#795)

* disable autoOptimizeShuffle

* re-enabled optimizeShuffleParts

* fixed exists tests (#799)

* Enable API Error Reporting to PipReport and Safe Failure (#796)

* possible solution

* complete implementation

* improved runtime calcs

* improved fail message stdout

---------

Co-authored-by: sourav.banerjee <sourav.banerjee@databricks.com>
aman-db pushed a commit that referenced this pull request Mar 16, 2023
* disable autoOptimizeShuffle

* re-enabled optimizeShuffleParts
aman-db pushed a commit that referenced this pull request Mar 28, 2023
* disable autoOptimizeShuffle

* re-enabled optimizeShuffleParts
sudharshanraja-db pushed a commit that referenced this pull request Apr 11, 2023
* initial commit

* 788 bug aws single tenant multiworkspace deployment noneget (#798)

* Change Validate for MWS logic

* removed debug print

---------

Co-authored-by: sourav.banerjee <sourav.banerjee@databricks.com>

* disable autoOptimizeShuffle (#795)

* disable autoOptimizeShuffle

* re-enabled optimizeShuffleParts

* fixed exists tests (#799)

* Enable API Error Reporting to PipReport and Safe Failure (#796)

* possible solution

* complete implementation

* improved runtime calcs

* improved fail message stdout

---------

Co-authored-by: sourav.banerjee <sourav.banerjee@databricks.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.

[PERF] - autoOptimizeShuffle was enabled and is causing perf issues
3 participants