-
Notifications
You must be signed in to change notification settings - Fork 436
Extract refresh support #159
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
Extract refresh support #159
Conversation
Part two of #125 This backfills all existing APIs with their minimum version. Checking this in early in release cycle for baketime.
* Add a new decorator that checks parameters against the version. Used with the @api decorator. * Add extract_only flags to workbooks and data sources, protected behind v2.5 flag
…d sample to download_view_image.py. Comments clean up
samples/refresh_tasks.py
Outdated
@@ -0,0 +1,71 @@ | |||
#### | |||
# TODO |
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.
dang it, I need to write the documentation for this. Coming soon
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.
🚀
Two minor things
@@ -33,6 +33,10 @@ def __init__(self, name, priority, schedule_type, execution_order, interval_item | |||
self.priority = priority | |||
self.schedule_type = schedule_type | |||
|
|||
def __repr__(self): | |||
print(self.__dict__) |
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.
Print?
return list(all_tasks) | ||
|
||
@classmethod | ||
def _parse_task(cls, element): |
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.
_parse_element
Hey @RussTheAerialist This looks great, thanks for working on the extract refresh feature! Is this safe to be used in production already? If not, do you have any ETA? Thanks again! |
Hi @ferkaz, this feature is slated to be a future release of the Tableau Server. It's not guaranteed in the next release, but should be there. Oh, and now that I look at it, I forgot to tag them with the rest api version, so I need to go a new pull request for that! |
Sounds good, thank you! |
* Response to code reviews. Put all request options into 1 file. renamed sample to download_view_image.py. Comments clean up * Add api annotation to all current endpoints (tableau#125) Part two of tableau#125 This backfills all existing APIs with their minimum version. Checking this in early in release cycle for baketime. * initial implement of get all and get specific for Extract Refresh Tasks * fixing test failure in the schedule_item code * pep8 fixes * Download with extract_only and parameter checking (tableau#143) * Add a new decorator that checks parameters against the version. Used with the @api decorator. * Add extract_only flags to workbooks and data sources, protected behind v2.5 flag * Correct the path to extract refresh tasks * fixing missed pep8 failure * adding runNow to the interface * fixing pep8 issues * Add header documentation to the sample. * addressing tyler's feedback
Hi all, I tried to run the file and got the error: "refresh_tasks.py: error: too few arguments" when I called: |
Can you give us the full call stack that happens? Also, which version of
TSC are you using?
…-Russell
On September 27, 2017 at 2:37:43 AM, dungkr89 ***@***.***) wrote:
Hi all, I tried to run the file and got the error: "refresh_tasks.py:
error: too few arguments" when I called:
python samples\refresh_tasks.py -s "http://server.it" -u "my name" -S
"Default"
Any idea? Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#159 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAFxVXJYyoeEBZyb6m_5VmdrfNeo8k1iks5smhdngaJpZM4MnWWX>
.
|
Oh! okay, I see what you mean. So the refresh tasks sample has three "commands" built into it. You can list all of the refresh tasks, you can run one of the refresh tasks, or you get information about the refresh task. In your case, if you run it like the following, it should work:
list will output a list of tasks, and you can then call info with the id of the task to get more details about it. Once you've identified which one you want to run, you can then use the run command to kick it off. |
@RussTheAerialist Thank you, it works well |
If you are getting a sign in error either you typed the password wrong, or you don't have access to the site you are trying to login to. If you are logging into the default site, you don't need to specify it on the command line. Try removing the |
Thank you @RussTheAerialist, I have moved to other project so I havent seen for some days. I have just tested it and it work as your suggestion. It's very nice experience, thank you again ;) |
Okay, that's better.