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: fix Guava version issue in Azure Synapse and Databricks #1103

Merged
merged 12 commits into from Jul 8, 2021

Conversation

xuwq1993
Copy link
Contributor

@xuwq1993 xuwq1993 commented Jun 25, 2021

Azure Synapse updated Guava to version 28.0

MoreExecutors.sameExecutorService() are not supported since Guava version >= 18
we should use this method: MoreExecutors.newDirectExecutorService()

but databricks use Guava version < 18, which don't contains MoreExecutors.newDirectExecutorService().

Therefore, I used reflection to load class "MoreExecutor" and try to get the correct Method in different environment with different Guava version.

@xuwq1993
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codecov
Copy link

codecov bot commented Jun 25, 2021

Codecov Report

Merging #1103 (d6781f0) into master (115f921) will increase coverage by 0.00%.
The diff coverage is 88.88%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1103   +/-   ##
=======================================
  Coverage   85.71%   85.72%           
=======================================
  Files         252      252           
  Lines       11597    11605    +8     
  Branches      593      599    +6     
=======================================
+ Hits         9940     9948    +8     
  Misses       1657     1657           
Impacted Files Coverage Δ
...icrosoft/ml/spark/automl/TuneHyperparameters.scala 79.01% <88.88%> (+0.93%) ⬆️
...com/microsoft/ml/spark/core/utils/AsyncUtils.scala 70.00% <0.00%> (+5.00%) ⬆️

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 115f921...d6781f0. Read the comment docs.

@xuwq1993
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xuwq1993
Copy link
Contributor Author

xuwq1993 commented Jul 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xuwq1993
Copy link
Contributor Author

xuwq1993 commented Jul 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xuwq1993
Copy link
Contributor Author

xuwq1993 commented Jul 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xuwq1993
Copy link
Contributor Author

xuwq1993 commented Jul 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xuwq1993
Copy link
Contributor Author

xuwq1993 commented Jul 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xuwq1993 xuwq1993 changed the title fix: update Guava version fix: fix Guava version issue in Synapse and Databricks Jul 8, 2021
@xuwq1993 xuwq1993 changed the title fix: fix Guava version issue in Synapse and Databricks fix: fix Guava version issue in Azure Synapse and Databricks Jul 8, 2021
@xuwq1993
Copy link
Contributor Author

xuwq1993 commented Jul 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xuwq1993
Copy link
Contributor Author

xuwq1993 commented Jul 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xuwq1993
Copy link
Contributor Author

xuwq1993 commented Jul 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@xuwq1993
Copy link
Contributor Author

xuwq1993 commented Jul 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mhamilton723 mhamilton723 merged commit fe70f31 into microsoft:master Jul 8, 2021
@xuwq1993 xuwq1993 deleted the wenqx/updateGuava branch July 13, 2021 02:55
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.

None yet

2 participants