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

Create rule S7192: The "how" parameter should be specified when joining two PySpark DataFrames #4634

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions rules/S7192/metadata.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
{
}
25 changes: 25 additions & 0 deletions rules/S7192/python/metadata.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
{
"title": "The \"how\" parameter should be specified when joining two PySpark DataFrames",
"type": "CODE_SMELL",
"status": "ready",
"remediation": {
"func": "Constant\/Issue",
"constantCost": "5min"
},
"tags": [
"data-science",
"pyspark"
],
"defaultSeverity": "Major",
"ruleSpecification": "RSPEC-7192",
"sqKey": "S7192",
"scope": "All",
"defaultQualityProfiles": ["Sonar way"],
"quickfix": "unknown",
"code": {
"impacts": {
"RELIABILITY": "MEDIUM"
},
"attribute": "CONVENTIONAL"
}
}
62 changes: 62 additions & 0 deletions rules/S7192/python/rule.adoc
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
This rule raises an issue when the parameter `how` is not provided when joining two PySpark `DataFrames`.

== Why is this an issue?

In PySpark, when joining two DataFrames, the `how` parameter specifies the type of join operation that will be performed. This parameter is crucial because it determines how the rows from the two DataFrames are combined based on the specified join condition. Common types of join operations include inner join, left join, right join, and outer join.

If the `how` parameter is not provided, PySpark will default to an inner join. This can lead to unexpected results if the join condition is not met, as rows that do not satisfy the join condition will be excluded from the result.

Specifying the `how` parameter explicitly is important because it defines the logic of how you want to combine the data from the two DataFrames. Depending on your data analysis needs, you might require different types of joins to get the desired results. For example, if you want to include all records from one DataFrame regardless of whether they have a match in the other, you would use a left or right outer join. If you only want records that have matches in both DataFrames, an inner join would be appropriate.

== How to fix it

To fix this issue, make sure to explicitly provide the `how` parameter when joining PySpark `DataFrames`.

=== Code examples

==== Noncompliant code example

[source,python,diff-id=1,diff-type=noncompliant]
----
# Joining DataFrames without specifying the 'how' parameter
df1 = spark.createDataFrame([(1, 'Alice'), (2, 'Bob')], ["id", "name"])
df2 = spark.createDataFrame([(1, 'HR'), (2, 'Finance')], ["id", "department"])

# Non-compliant: 'how' parameter is not specified
joined_df = df1.join(df2, on="id")
----

==== Compliant solution

[source,python,diff-id=1,diff-type=compliant]
----
# Joining DataFrames with the 'how' parameter specified
df1 = spark.createDataFrame([(1, 'Alice'), (2, 'Bob')], ["id", "name"])
df2 = spark.createDataFrame([(1, 'HR'), (2, 'Finance')], ["id", "department"])

# Compliant: 'how' parameter is specified
joined_df = df1.join(df2, on="id", how="inner")
----

== Resources
=== Documentation
* Spark Documentation - https://spark.apache.org/docs/latest/api/python/reference/pyspark.sql/api/pyspark.sql.DataFrame.join.html[DataFrame.join]

=== Related rules
- S6735 - When using pandas.merge or pandas.join, the parameters on, how and validate should be provided

ifdef::env-github,rspecator-view[]

'''
== Implementation Specification
(visible only on this page)

=== Message

Specify the `how` parameter of this join.

=== Quickfix

Add the `how` parameter to the join operation, with the default value set to `"inner"`.

endif::env-github,rspecator-view[]