Skip to content

Commit 973bd9c

Browse files
authored
feat(6483): Separate queries and results, for graph versus table. (#6484)
* chore: results context is a context, not a component. * feat(6483): make a singular resultsView, whereas we have a separate results context (which we can then have multiple) * feat(6483): disentangle the TableResults view from the GraphResults view * feat(6483): have childResults view subscribe to parent results, such that an graph related subqueries are overwritten with changes to editor query. * feat(6483): add viewOptions, and have childResults subscribe to viewOptions such that a subquery is issued if needed. * fix(6483): ChildResultsContext subscribes to PersistanceContext, therefore need to nest correctly. Trigger graph subquery on change from parent, as noticed via results change (not queryText change -- as that will be triggered too often). * feat(6483): modify sqlQuery based on SqlQueryModifiers, in the query context * chore(6483): cleanup types, naming conventions, use of placeholder components. Reduce re-renders. Clarify conditional expression.
1 parent 0990ccd commit 973bd9c

File tree

13 files changed

+432
-199
lines changed

13 files changed

+432
-199
lines changed

src/dataExplorer/components/DeleteScript.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import {useSelector, useDispatch} from 'react-redux'
88
import {getOrg} from 'src/organizations/selectors'
99
import {notify} from 'src/shared/actions/notifications'
1010
import {deleteScriptFail} from 'src/shared/copy/notifications/categories/scripts'
11-
import {ResultsContext} from 'src/dataExplorer/components/ResultsContext'
11+
import {ResultsContext} from 'src/dataExplorer/context/results'
1212
import {RemoteDataState} from 'src/types'
1313
import {QueryContext} from 'src/shared/contexts/query'
1414
import {SCRIPT_EDITOR_PARAMS} from 'src/dataExplorer/components/resources'

src/dataExplorer/components/FluxQueryBuilder.tsx

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,9 @@ import {
1919
} from '@influxdata/clockface'
2020
import {QueryProvider} from 'src/shared/contexts/query'
2121
import {EditorProvider} from 'src/shared/contexts/editor'
22-
import {ResultsProvider} from 'src/dataExplorer/components/ResultsContext'
22+
import {ResultsProvider, ResultsContext} from 'src/dataExplorer/context/results'
23+
import {ChildResultsProvider} from 'src/dataExplorer/context/results/childResults'
24+
import {ResultsViewProvider} from 'src/dataExplorer/context/resultsView'
2325
import {SidebarProvider} from 'src/dataExplorer/context/sidebar'
2426
import {
2527
PersistanceProvider,
@@ -31,7 +33,6 @@ import Sidebar from 'src/dataExplorer/components/Sidebar'
3133
import Schema from 'src/dataExplorer/components/Schema'
3234
import SaveAsScript from 'src/dataExplorer/components/SaveAsScript'
3335
import {QueryContext} from 'src/shared/contexts/query'
34-
import {ResultsContext} from 'src/dataExplorer/components/ResultsContext'
3536
import {getOrg, isOrgIOx} from 'src/organizations/selectors'
3637
import {RemoteDataState} from 'src/types'
3738
import {SCRIPT_EDITOR_PARAMS} from 'src/dataExplorer/components/resources'
@@ -222,9 +223,13 @@ const FluxQueryBuilder: FC = () => {
222223
export default () => (
223224
<QueryProvider>
224225
<ResultsProvider>
225-
<PersistanceProvider>
226-
<FluxQueryBuilder />
227-
</PersistanceProvider>
226+
<ResultsViewProvider>
227+
<PersistanceProvider>
228+
<ChildResultsProvider>
229+
<FluxQueryBuilder />
230+
</ChildResultsProvider>
231+
</PersistanceProvider>
232+
</ResultsViewProvider>
228233
</ResultsProvider>
229234
</QueryProvider>
230235
)

src/dataExplorer/components/OpenScript.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ import {CLOUD} from 'src/shared/constants'
1717
import {useHistory} from 'react-router-dom'
1818
import {useSelector} from 'react-redux'
1919
import {getOrg} from 'src/organizations/selectors'
20-
import {ResultsContext} from 'src/dataExplorer/components/ResultsContext'
20+
import {ResultsContext} from 'src/dataExplorer/context/results'
2121
import {QueryContext} from 'src/shared/contexts/query'
2222
import {debouncer} from 'src/dataExplorer/shared/utils'
2323

src/dataExplorer/components/QueryTime.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import React, {FC, useContext} from 'react'
2-
import {ResultsContext} from 'src/dataExplorer/components/ResultsContext'
2+
import {ResultsContext} from 'src/dataExplorer/context/results'
33
import {RemoteDataState} from 'src/types'
44

55
import 'src/dataExplorer/components/QueryTime.scss'

0 commit comments

Comments
 (0)