Skip to content

Conversation

@panbingkun
Copy link
Contributor

What changes were proposed in this pull request?

The pr aims to implement SparkSession.version and SparkSession.time.

Why are the changes needed?

API coverage.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Add new UT.

Copy link
Contributor

@hvanhovell hvanhovell left a comment

Choose a reason for hiding this comment

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

LGTM

@hvanhovell
Copy link
Contributor

Merging

hvanhovell pushed a commit that referenced this pull request Feb 27, 2023
…n.time

### What changes were proposed in this pull request?
The pr aims to implement SparkSession.version and SparkSession.time.

### Why are the changes needed?
API coverage.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Add new UT.

Closes #40176 from panbingkun/SPARK-42564.

Authored-by: panbingkun <pbk1982@gmail.com>
Signed-off-by: Herman van Hovell <herman@databricks.com>
(cherry picked from commit 9c7aa16)
Signed-off-by: Herman van Hovell <herman@databricks.com>
@hvanhovell
Copy link
Contributor

Thanks for doing this!

@dongjoon-hyun
Copy link
Member

Thank you for doing this too!

snmvaughan pushed a commit to snmvaughan/spark that referenced this pull request Jun 20, 2023
…n.time

### What changes were proposed in this pull request?
The pr aims to implement SparkSession.version and SparkSession.time.

### Why are the changes needed?
API coverage.

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Add new UT.

Closes apache#40176 from panbingkun/SPARK-42564.

Authored-by: panbingkun <pbk1982@gmail.com>
Signed-off-by: Herman van Hovell <herman@databricks.com>
(cherry picked from commit 9c7aa16)
Signed-off-by: Herman van Hovell <herman@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants