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

[SPARK-5722] [SQL] [PySpark] infer int as LongType in Python (for 1.2 branch) #4681

Closed
wants to merge 1 commit into from

Conversation

davies
Copy link
Contributor

@davies davies commented Feb 18, 2015

This PR change to use LongType for int in Python, when inferSchema(), because IntegerType in SQL is not enough for int in Python (which is 64-bit on 64-bit machines).

Closes #4521

cc @dondrake @marmbrus

@SparkQA
Copy link

SparkQA commented Feb 18, 2015

Test build #27694 has started for PR 4681 at commit 05ef1c8.

  • This patch merges cleanly.

@davies davies changed the title [SPARK-5722] [SQL] [PySpark] infer int as LongType in Python [SPARK-5722] [SQL] [PySpark] infer int as LongType in Python (for 1.2 branch) Feb 18, 2015
@SparkQA
Copy link

SparkQA commented Feb 19, 2015

Test build #27694 has finished for PR 4681 at commit 05ef1c8.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27694/
Test PASSed.

@dondrake
Copy link

@davies @marmbrus Just curious what the status of this PR and JIRA are for 1.2.x?

Thanks.

asfgit pushed a commit that referenced this pull request Feb 24, 2015
… branch)

This PR change to use LongType for int in Python, when inferSchema(), because IntegerType in SQL is not enough for int in Python (which is 64-bit on 64-bit machines).

Closes #4521

cc dondrake marmbrus

Author: Davies Liu <davies@databricks.com>

Closes #4681 from davies/long2 and squashes the following commits:

05ef1c8 [Davies Liu] infer LongType for int in Python
@marmbrus
Copy link
Contributor

Merged to branch-1.2

@davies can you close this issue?

@davies
Copy link
Contributor Author

davies commented Feb 24, 2015

Thanks!

@davies davies closed this Feb 24, 2015
markhamstra pushed a commit to markhamstra/spark that referenced this pull request Feb 24, 2015
… branch)

This PR change to use LongType for int in Python, when inferSchema(), because IntegerType in SQL is not enough for int in Python (which is 64-bit on 64-bit machines).

Closes apache#4521

cc dondrake marmbrus

Author: Davies Liu <davies@databricks.com>

Closes apache#4681 from davies/long2 and squashes the following commits:

05ef1c8 [Davies Liu] infer LongType for int in Python
sunchao pushed a commit to sunchao/spark that referenced this pull request Jun 2, 2023
PRs Merged
1. [Internal] Add AppleAwsClientFactory for Mascot (apache#577)
2. Hive: Log new metadata location in commit (apache#4681)
3. change timeout to 120 for now (apache#661)
4. Internal: Add hive_catalog parameter to SparkCatalog (apache#670)
5. Internal: Pull catalog setting to CachedClientPool (apache#673)
6. Core: Defer reading Avro metadata until ManifestFile is read (apache#5206)
7. API: Fix ID assignment in schema merging (apache#5395)
8. AWS: S3OutputStream - failure to close should persist on subsequent close calls (apache#5311)
9. API: Allow schema updates to find fields with case-insensitivity (apache#5440)
10. Spark 3.3: Spark mergeSchema to respect Spark Case Sensitivity Configuration (apache#5441)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants