Skip to content

Conversation

@atzoum
Copy link
Collaborator

@atzoum atzoum commented Aug 29, 2024

Description

  • Removed unnecessary IncludesViewsInListTables flag from integration tests
  • Added a scenario for verifying that view's columns are returned during ListColumns as well

Security

  • The code changed/added as part of this pull request won't create any security issues with how the software is being used.

@atzoum atzoum force-pushed the fix.listcolumns branch 2 times, most recently from f87f899 to d90bbbc Compare August 29, 2024 08:41
@atzoum atzoum requested a review from shubhammehra4 August 29, 2024 08:46
@atzoum atzoum marked this pull request as ready for review August 29, 2024 08:47
@codecov
Copy link

codecov bot commented Aug 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.58%. Comparing base (5f799ab) to head (3fa0e0f).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #169      +/-   ##
==========================================
- Coverage   85.58%   85.58%   -0.01%     
==========================================
  Files          78       78              
  Lines        3330     3343      +13     
==========================================
+ Hits         2850     2861      +11     
- Misses        327      328       +1     
- Partials      153      154       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@atzoum atzoum merged commit 098a378 into main Aug 29, 2024
@atzoum atzoum deleted the fix.listcolumns branch August 29, 2024 09:13
atzoum pushed a commit that referenced this pull request Sep 12, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.10.0](v1.9.0...v1.10.0)
(2024-09-12)


### Features

* **databricks:** bump github.com/databricks/databricks-sql-go from
1.6.0 to 1.6.1
([#174](#174))
([8cab1fd](8cab1fd))
* **snowflake:** bump github.com/snowflakedb/gosnowflake from 1.10.1 to
1.11.1 ([#171](#171))
([e64ba77](e64ba77))
* support dialect-aware query conditions
([#180](#180))
([08906ee](08906ee))


### Miscellaneous

* all warehouses support views in ListTables
([#169](#169))
([098a378](098a378))
* **deps:** bump github.com/aws/aws-sdk-go-v2/credentials from 1.17.27
to 1.17.32
([#179](#179))
([47afb2e](47afb2e))
* **deps:** bump github.com/rudderlabs/rudder-go-kit from 0.36.2 to
0.40.0 ([#178](#178))
([a4ee52e](a4ee52e))
* **deps:** bump google.golang.org/api from 0.194.0 to 0.196.0
([#177](#177))
([aaee20d](aaee20d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants