dgd-contributor commented on a change in pull request #33911: URL: https://github.com/apache/spark/pull/33911#discussion_r704017833
########## File path: python/pyspark/pandas/data_type_ops/boolean_ops.py ########## @@ -248,6 +249,30 @@ def and_func(left: Column, right: Any) -> Column: return column_op(and_func)(left, right) + def __xor__(self, left: IndexOpsLike, right: Any) -> SeriesOrIndex: + _sanitize_list_like(right) + if isinstance(right, IndexOpsMixin) and isinstance(right.dtype, extension_dtypes): + return right.__xor__(left) + elif ( + isinstance(right, int) + or (hasattr(right, "dtype") and np.issubdtype(right.dtype, int)) + or isinstance(right, bool) + or (hasattr(right, "dtype") and right.dtype == "bool") Review comment: Thank you Yeah, they have same result. But it throw "Circular import" when I try to extract that ```elif``` to ```data_type_ops/base.py```. I do not understand ```IndexOpsMixin```, I am appreciated if you could explain it and the better point. I deduplicated code. Could you take another look? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org