Skip to content

Commit

Permalink
Prepare for 0.7.3 release (#367)
Browse files Browse the repository at this point in the history
* Add release note

* update
  • Loading branch information
goodwanghan authored Oct 3, 2022
1 parent c7bc70f commit 5d887ba
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 2 additions & 0 deletions RELEASE.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@
- [362](https://github.com/fugue-project/fugue/issues/362) Remove Python 3.6 Support
- [363](https://github.com/fugue-project/fugue/issues/363) Create IbisDataFrame and IbisExecutionEngine
- [364](https://github.com/fugue-project/fugue/issues/364) Enable Map type support
- [365](https://github.com/fugue-project/fugue/issues/365) Support column names starting with numbers
- [361](https://github.com/fugue-project/fugue/issues/361) Better error message for cross join

## 0.7.2

Expand Down
7 changes: 4 additions & 3 deletions fugue/dataframe/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -490,10 +490,11 @@ def get_join_schemas(
)
cm = df1.schema.intersect(on)
if how == "cross":
cs = df1.schema.intersect(schema2)
aot(
len(df1.schema.intersect(schema2)) == 0,
SchemaError("can't specify on for cross join"),
len(cs) == 0,
SchemaError(f"invalid cross join, two dataframes have common columns {cs}"),
)
else:
aot(len(on) > 0, SchemaError("on must be specified"))
aot(len(on) > 0, SchemaError("join on columns must be specified"))
return cm, (df1.schema.union(schema2))

0 comments on commit 5d887ba

Please sign in to comment.