Skip to content

fix(nativeFilters): Speed up native filters by removing unnecessary rerenders #25282

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.

Sign up for GitHub

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

Merged
kgabryje merged 12 commits into from
Sep 28, 2023
+16 −5
Merged

fix(nativeFilters): Speed up native filters by removing unnecessary rerenders #25282

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
08305d7
feat(nativeFilters): optimize filters rendering by adding __cache to …
Always-prog Sep 13, 2023
e6fb569
lint fix
Always-prog Sep 13, 2023
64b712c
removed surplus JSON.stringify for dataMask
Always-prog Sep 13, 2023
6a361ae
using ref for dataMask instead of hooking by JSON.stringify
Always-prog Sep 14, 2023
32c7048
OOPS
Always-prog Sep 14, 2023
742fb47
dataMask ref instead of JSON.stringify in Vertical filters
Always-prog Sep 14, 2023
2bc1250
lint fix
Always-prog Sep 14, 2023
d4217bf
removed using ref in vertical filter bar, only in parent - filterBar
Always-prog Sep 14, 2023
1e708ce
update dataMaskSelectedRef sync logic
Always-prog Sep 19, 2023
5d85a0d
return back dataMaskSelected without JSON.strignifying
Always-prog Sep 23, 2023
7e52a69
lint fix
Always-prog Sep 24, 2023
84ba073
fix(native-filters): added shallowEqual for useFilterDependencies sel…
Always-prog Sep 27, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions superset-frontend/packages/superset-ui-core/src/chart/types/Base.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,7 @@ export enum AppSection {
export type FilterState = { value?: any; [key: string]: any };

export type DataMask = {
__cache?: FilterState;
extraFormData?: ExtraFormData;
filterState?: FilterState;
ownState?: JsonObject;
Expand Down
7 changes: 6 additions & 1 deletion superset-frontend/src/dashboard/components/nativeFilters/FilterBar/Vertical.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -220,7 +220,12 @@ const VerticalFilterBar: React.FC<VerticalBarProps> = ({
/>
</FilterControlsWrapper>
),
[canEdit, dataMaskSelected, filterValues.length, onSelectionChange],
[
canEdit,
JSON.stringify(dataMaskSelected),
filterValues.length,
onSelectionChange,
],
);

const filterSetsTabs = useMemo(
Expand Down
3 changes: 2 additions & 1 deletion superset-frontend/src/dashboard/components/nativeFilters/FilterBar/index.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,7 @@ const FilterBar: React.FC<FiltersBarProps> = ({
verticalConfig,
hidden = false,
}) => {
console.log('rerender');
Copy link
Member

@villebro villebro Sep 13, 2023

Choose a reason for hiding this comment

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

Oops

Suggested change
console.log('rerender');
console.log('rerender');

const history = useHistory();
const dataMaskApplied: DataMaskStateWithId = useNativeFiltersDataMask();
const [dataMaskSelected, setDataMaskSelected] =
Expand Down Expand Up @@ -166,7 +167,7 @@ const FilterBar: React.FC<FiltersBarProps> = ({
};
});
},
[dataMaskSelected, dispatch, setDataMaskSelected],
[JSON.stringify(dataMaskSelected), dispatch, setDataMaskSelected],
);

useEffect(() => {
Expand Down
1 change: 1 addition & 0 deletions superset-frontend/src/dataMask/reducer.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ export function getInitialDataMask(
}
return {
...otherProps,
__cache: {},
extraFormData: {},
filterState: {},
ownState: {},
Expand Down