Skip to content

add support for warehouse schemas#393

Merged
martiner merged 1 commit intogooddata:masterfrom
Tantalor93:obe-schemas
Nov 1, 2016
Merged

add support for warehouse schemas#393
martiner merged 1 commit intogooddata:masterfrom
Tantalor93:obe-schemas

Conversation

@Tantalor93
Copy link
Contributor

@Tantalor93 Tantalor93 commented Oct 29, 2016

No description provided.

private Map<String, String> links;

@JsonCreator
public WarehouseSchema(@JsonProperty("name") String name,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

private?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed to private

public class WarehouseServiceIT extends AbstractGoodDataIT {

private static final String TITLE = "Test";
private static final String SCHEMA_NAME = "default";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the default schema? is it something always present? maybe user would use some handy get method in the service?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, default schema always exists in warehouse. Added method getDefaultWarehouseSchema to the service.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 82.535% when pulling d3927c9 on Tantalor93:obe-schemas into 714a764 on martiner:master.

@martiner martiner added this to the next milestone Nov 1, 2016
public static final String URI = WarehouseSchemas.URI + "/{name}";
public static final UriTemplate TEMPLATE = new UriTemplate(URI);

private String name;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

final?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 82.535% when pulling 3d46c8a on Tantalor93:obe-schemas into 7ee9e22 on martiner:master.

@martiner martiner merged commit bedd7de into gooddata:master Nov 1, 2016
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.

3 participants