-
Notifications
You must be signed in to change notification settings - Fork 62
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
Introduce Warehouse and Dataload Processes API support #48
Conversation
please rename ads to warehouse |
|
||
@Override | ||
protected boolean isFinished(ClientHttpResponse response) throws IOException { | ||
return HttpStatus.CREATED.equals(response.getStatusCode()); |
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.
add check the instance is enabled
2938ffb
to
991bb0c
Compare
* @param project project to which the process belongs | ||
* @return created process | ||
*/ | ||
public Process createProcess(Process process, File processData, Project project) { |
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.
project should be the first argument
*/ | ||
public ProcessService(RestTemplate restTemplate, AccountService accountService) { | ||
super(restTemplate); | ||
this.accountService = accountService; |
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.
not null
@Override | ||
protected void onFinish() { | ||
if (!getResult().isEnabled()) { | ||
throw new GoodDataException("Created warehouse is not enabled!"); |
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.
please add ware house uri to the message
Introduce Warehouse and Dataload Processes API support
No description provided.