Moved ajax call for fetching table metadata from SqlEditorLeftBar to actions (#1494)

This commit is contained in:
vera-liu
2016-11-01 09:00:24 -07:00
committed by GitHub
parent 2fd2526046
commit 52380534f3
2 changed files with 38 additions and 29 deletions

View File

@@ -87,37 +87,10 @@ class SqlEditorLeftBar extends React.PureComponent {
changeTable(tableOpt) {
const tableName = tableOpt.value;
const qe = this.props.queryEditor;
let url = `/caravel/table/${qe.dbId}/${tableName}/${qe.schema}/`;
this.setState({ tableLoading: true });
$.get(url, (data) => {
this.props.actions.mergeTable(Object.assign(data, {
dbId: qe.dbId,
queryEditorId: qe.id,
schema: qe.schema,
expanded: true,
}));
this.setState({ tableLoading: false });
})
.fail(() => {
this.props.actions.addAlert({
msg: 'Error occurred while fetching metadata',
bsStyle: 'danger',
});
this.setState({ tableLoading: false });
});
url = `/caravel/extra_table_metadata/${qe.dbId}/${tableName}/${qe.schema}/`;
$.get(url, (data) => {
const table = {
dbId: this.props.queryEditor.dbId,
queryEditorId: this.props.queryEditor.id,
schema: qe.schema,
name: tableName,
};
Object.assign(table, data);
this.props.actions.mergeTable(table);
});
this.props.actions.addTable(qe, tableName);
this.setState({ tableLoading: false });
}
render() {
let networkAlert = null;