Tagging @WeichenXu123, since you reviewed the most recent MR.

@ttc0419 Thank you! I will take a look soon.

The trait GraphFrameTestSparkContext still has code protected override def _sqlContext: SQLContext = self.sqlContext, can we replace it with SparkSession ?

The trait GraphFrameTestSparkContext still has code protected override def _sqlContext: SQLContext = self.sqlContext, can we replace it with SparkSession ?

I'm not familiar with Scala, probably need someone else and another MR to do that.

The trait GraphFrameTestSparkContext still has code protected override def _sqlContext: SQLContext = self.sqlContext, can we replace it with SparkSession ?

I'm not familiar with Scala, probably need someone else and another MR to do that.

We can do it in follow-up PR.

@WeichenXu123 Anything else need to be done before merging this?

Codecov Report

Merging #424 (59f5bd0) into master (100fb01) will decrease coverage by 0.19%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##           master     #424      +/-   ##
==========================================
- Coverage   91.37%   91.19%   -0.19%     
==========================================
  Files          18       18              
  Lines         870      829      -41     
  Branches       53       49       -4     
==========================================
- Hits          795      756      -39     
+ Misses         75       73       -2     

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

@WeichenXu123 Do we need another reviewer to merge this?

@WeichenXu123 Do we need another reviewer to merge this?

I can merge it.