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
Renamed size_utils.sql #5368
Renamed size_utils.sql #5368
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These names are kind of confusing :) I'd suggest size_utils_apache
and size_utils_tsl
.
Agree that the current renaming is a bit confusing but would like to simplify a bit it: |
Codecov Report
@@ Coverage Diff @@
## main #5368 +/- ##
==========================================
- Coverage 90.69% 90.68% -0.02%
==========================================
Files 226 226
Lines 52522 52519 -3
==========================================
- Hits 47635 47627 -8
- Misses 4887 4892 +5
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Renamed: tsl/test/sql/size_utils.sql tsl/test/expected/size_utils.out To: tsl/test/sql/size_utils_tsl.sql tsl/test/expected/size_utils_tsl.out because conflicting with test/sql/size_utils.sql
@akuzm, @mkindahl: please review this pull request.
|
Renamed test/sql/size_utils.sql
to test/sql/utils_size.sql
because conflicting with tsl/test/sql/size_utils.sql