Skip to content
This repository has been archived by the owner on Mar 30, 2021. It is now read-only.

Append token= to CREDENTIAL param of COPY command #13

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tkawachi
Copy link
Contributor

@tkawachi tkawachi commented Apr 8, 2014

No description provided.

@tkawachi
Copy link
Contributor Author

Thanks for pointing out. I did it.

@tkawachi
Copy link
Contributor Author

@gauravgh Could you review this pr?

@akshah123
Copy link

Hi @tkawachi , are you running into a problem where copy commands start failing after few hours? If so, is this because tokens are generated in constructor and not in the emit function?

@tkawachi
Copy link
Contributor Author

It should be. I was not aware because I'm not actually using this class in
production. I'll modify the pull request later. Thanks.

2014年4月22日火曜日、akshah123notifications@github.comさんは書きました:

Hi @tkawachi https://github.com/tkawachi , are you running into a
problem where copy commands start failing after few hours? If so, is this
because tokens are generated in constructor and not in the emit function?


Reply to this email directly or view it on GitHubhttps://github.com//pull/13#issuecomment-40948990
.

モバイル Gmail から送信

@chrisferry
Copy link

👍 Any reason why this is taking so long to merge into master?

@pmvilaca
Copy link

pmvilaca commented Jun 7, 2016

+1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants