Skip to content

Conversation

WeichenXu123
Copy link
Contributor

Remove deprecated sqlContext in graphframe scala code

Signed-off-by: Weichen Xu <weichen.xu@databricks.com>
Signed-off-by: Weichen Xu <weichen.xu@databricks.com>
Signed-off-by: Weichen Xu <weichen.xu@databricks.com>
Signed-off-by: Weichen Xu <weichen.xu@databricks.com>
@codecov-commenter
Copy link

codecov-commenter commented Jul 18, 2023

Codecov Report

Merging #432 (9af257d) into master (3d76998) will increase coverage by 0.23%.
The diff coverage is 90.90%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master     #432      +/-   ##
==========================================
+ Coverage   91.07%   91.30%   +0.23%     
==========================================
  Files          18       18              
  Lines         829      828       -1     
  Branches       49       47       -2     
==========================================
+ Hits          755      756       +1     
+ Misses         74       72       -2     
Impacted Files Coverage Δ
...a/org/graphframes/examples/BeliefPropagation.scala 82.25% <0.00%> (+1.30%) ⬆️
.../scala/org/graphframes/lib/GraphXConversions.scala 84.61% <71.42%> (ø)
src/main/scala/org/graphframes/GraphFrame.scala 87.11% <100.00%> (ø)
...c/main/scala/org/graphframes/examples/Graphs.scala 100.00% <100.00%> (ø)
.../scala/org/graphframes/lib/AggregateMessages.scala 93.54% <100.00%> (ø)
src/main/scala/org/graphframes/lib/BFS.scala 100.00% <100.00%> (ø)
...cala/org/graphframes/lib/ConnectedComponents.scala 95.37% <100.00%> (+0.92%) ⬆️

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

@WeichenXu123 WeichenXu123 merged commit a2c1dbc into master Jul 18, 2023
@SemyonSinchenko SemyonSinchenko deleted the remove-sql-ctx branch June 10, 2025 09:01
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.

4 participants