search

GoogleCloudPlatform / bigquery-antipattern-recognition

Utility to identify and rewrite common anti patterns in BigQuery SQL syntax
Apache License 2.0
76 stars 26 forks source link

Remove outdated/obsolete anti pattern 2 #12

Closed danieldeleo closed 1 year ago

danieldeleo commented 1 year ago

Regarding "Anti Pattern 2: Using CROSS JOINs when INNER JOINs are an option", BigQuery now optimizes this into an inner join automatically. Recommend removing from the tool and docs

franklinWhaite commented 1 year ago

Thanks for bringing this up. That anti pattern is actually deprecated in the code since a while ago, we had not updated the readme, we have create a PR to fix this.

franklinWhaite commented 1 year ago

Readme updated