Skip to content

[SPARK-56954][PYTHON] Annotate that we support MapType for from_json#55997

Closed
gaogaotiantian wants to merge 2 commits into
apache:masterfrom
gaogaotiantian:from-json-map-type
Closed

[SPARK-56954][PYTHON] Annotate that we support MapType for from_json#55997
gaogaotiantian wants to merge 2 commits into
apache:masterfrom
gaogaotiantian:from-json-map-type

Conversation

@gaogaotiantian
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

  • Add annotation for from_json that we support MapType as schema input.
  • Clean up the docs so it's consistent.
  • Add a test for MapType.

Why are the changes needed?

We already supported the feature, just need the annotation so it's clear to the users and also type checkers won't complaint.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

CI for both lint and new test.

Was this patch authored or co-authored using generative AI tooling?

No.

zhengruifeng pushed a commit that referenced this pull request May 21, 2026
### What changes were proposed in this pull request?

* Add annotation for `from_json` that we support `MapType` as schema input.
* Clean up the docs so it's consistent.
* Add a test for `MapType`.

### Why are the changes needed?

We already supported the feature, just need the annotation so it's clear to the users and also type checkers won't complaint.

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

No.

### How was this patch tested?

CI for both lint and new test.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #55997 from gaogaotiantian/from-json-map-type.

Authored-by: Tian Gao <gaogaotiantian@hotmail.com>
Signed-off-by: Ruifeng Zheng <ruifengz@foxmail.com>
(cherry picked from commit 2ffd11d)
Signed-off-by: Ruifeng Zheng <ruifengz@foxmail.com>
@zhengruifeng
Copy link
Copy Markdown
Contributor

merged to master/4.x

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.

2 participants