Skip to content
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

<<Do not Merge>> Added additional logs for Snowflake. #7452

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

mkchiliveri
Copy link
Contributor

Change(s):

  • Added new additional logs related to Grants to User and Data transfer history

Reason for Change(s):

  • Feature request

Version Updated:

  • N/A

Testing Completed:

  • Yes

Checked that the validations are passing and have addressed any issues that are present:

  • Yes

@mkchiliveri mkchiliveri requested review from a team as code owners February 28, 2023 11:48
@v-atulyadav v-atulyadav added the Connector Connector specialty review needed label Feb 28, 2023
@v-sabiraj
Copy link
Contributor

@mkchiliveri, I think it will remain open till confirmations?

@v-sabiraj
Copy link
Contributor

@mkchiliveri, can we merge this?

@mkchiliveri mkchiliveri changed the title Added additional logs for Snowflake. --Do not Merge-- Added additional logs for Snowflake. Mar 6, 2023
@v-sabiraj
Copy link
Contributor

@mkchiliveri, can you please confirm if we can merge?

@v-sabiraj
Copy link
Contributor

@mkchiliveri, should this be merged?

@v-sabiraj
Copy link
Contributor

Waiting on merge confirmation

@v-prasadboke
Copy link
Contributor

Hello @v-sabiraj any updates on the following

@v-rbajaj
Copy link
Contributor

v-rbajaj commented Apr 4, 2023

Hi @v-sabiraj, can you please share your feedback?

@v-sabiraj
Copy link
Contributor

@v-rbajaj, still waiting for merge confirmation.

@v-prasadboke
Copy link
Contributor

Hello @v-sabiraj any updtes

@v-prasadboke
Copy link
Contributor

Hello @v-sabiraj pr is approve can you please provide feedback

1 similar comment
@v-atulyadav
Copy link
Contributor

Hello @v-sabiraj pr is approve can you please provide feedback

@v-atulyadav
Copy link
Contributor

Waiting on merge confirmation

@v-prasadboke
Copy link
Contributor

Waiting for merge confirmation

@v-prasadboke
Copy link
Contributor

Hello @v-sabiraj waiting for your reply

@v-prasadboke
Copy link
Contributor

Hello @mkchiliveri waiting for merge confirmation

@v-atulyadav v-atulyadav marked this pull request as draft June 16, 2023 12:38
@v-amolpatil v-amolpatil changed the title --Do not Merge-- Added additional logs for Snowflake. <<Do not Merge>> Added additional logs for Snowflake. Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Connector Connector specialty review needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants