Skip to content
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

Maximum update depth exceeded with AutoGrid columnRendering set to lazy #255

Open
sagarbendale opened this issue Jun 20, 2024 · 1 comment

Comments

@sagarbendale
Copy link

I see this error on AutoGrid.
Whenever I have columnRendering set to lazy I see this error. If I change it back to eager this error will go away.

Warning: Maximum update depth exceeded. This can happen when a component calls setState inside useEffect, but useEffect either doesn’t have a dependency array, or one of the dependencies changes on every render.

console.error @ list:5
r9.error @ useRenderer.ts:60
printWarning @ react-dom.development.js:86
error @ react-dom.development.js:60
checkForNestedUpdates @ react-dom.development.js:27339
scheduleUpdateOnFiber @ react-dom.development.js:25514
dispatchReducerAction @ react-dom.development.js:16633
(anonymous) @ useRenderer.ts:62
flushSync @ react-dom.development.js:26228
flushSync$1 @ react-dom.development.js:29878
(anonymous) @ useRenderer.ts:62
_runRenderer @ vaadin-grid-column-mixin.js:612
(anonymous) @ vaadin-grid-column-mixin.js:643
__renderCellsContent @ vaadin-grid-column-mixin.js:626
_renderBodyCellsContent @ vaadin-grid-column-mixin.js:709
_onRendererOrBindingChanged @ vaadin-grid-column-mixin.js:714
runMethodEffect @ property-effects.js:1038
runEffects @ property-effects.js:140
_propertiesChanged @ property-effects.js:1922
_flushProperties @ properties-changed.js:384
_invalidateProperties @ property-effects.js:1748
Object.defineProperty.set @ properties-changed.js:170
(anonymous) @ vaadin-grid-mixin.js:547
_createScrollerRows @ vaadin-grid-mixin.js:545
_createPool @ virtualizer-iron-list-adapter.js:408
_increasePoolIfNeeded @ iron-list-core.js:434
_increasePoolIfNeeded @ virtualizer-iron-list-adapter.js:724
flush @ debounce.js:124
(anonymous) @ debounce.js:159
flushDebouncers @ debounce.js:157
flush @ debounce.js:172
set size @ virtualizer-iron-list-adapter.js:355
set size @ virtualizer.js:49
_flatSizeChanged @ vaadin-grid-mixin.js:300
runMethodEffect @ property-effects.js:1038
runEffects @ property-effects.js:140
_propertiesChanged @ property-effects.js:1922
_flushProperties @ properties-changed.js:384
_invalidateProperties @ property-effects.js:1748
Object.defineProperty.set @ properties-changed.js:170
_onDataProviderPageReceived @ vaadin-grid-data-provider-mixin.js:348
_onDataProviderPageReceived @ vaadin-grid-array-da…rovider-mixin.js:45
callback @ data-provider-controller.js:254
load @ data-provider.ts:91
await in load (async)
__loadCachePage @ data-provider-controller.js:265
loadFirstPage @ data-provider-controller.js:215
_ensureFirstPageLoaded @ vaadin-grid-data-provider-mixin.js:440
_dataProviderChanged @ vaadin-grid-data-provider-mixin.js:426
runObserverEffect @ property-effects.js:231
runEffects @ property-effects.js:140
_propertiesChanged @ property-effects.js:1922
_flushProperties @ properties-changed.js:384
_invalidateProperties @ property-effects.js:1748
Object.defineProperty.set @ properties-changed.js:170
(anonymous) @ autogrid.tsx:412
setTimeout (async)
(anonymous) @ autogrid.tsx:395
commitHookEffectListMount @ react-dom.development.js:23189
commitPassiveMountOnFiber @ react-dom.development.js:24965
commitPassiveMountEffects_complete @ react-dom.development.js:24930
commitPassiveMountEffects_begin @ react-dom.development.js:24917
commitPassiveMountEffects @ react-dom.development.js:24905
flushPassiveEffectsImpl @ react-dom.development.js:27078
flushPassiveEffects @ react-dom.development.js:27023
performSyncWorkOnRoot @ react-dom.development.js:26115
flushSyncCallbacks @ react-dom.development.js:12042
flushSync @ react-dom.development.js:26240
flushSync$1 @ react-dom.development.js:29878
(anonymous) @ useRenderer.ts:62
_runRenderer @ vaadin-grid-column-mixin.js:612
(anonymous) @ vaadin-grid-column-mixin.js:643
__renderCellsContent @ vaadin-grid-column-mixin.js:626
_renderHeaderCellContent @ vaadin-grid-column-mixin.js:672
_onHeaderRendererOrBindingChanged @ vaadin-grid-column-mixin.js:680
runMethodEffect @ property-effects.js:1038
runEffects @ property-effects.js:140
_propertiesChanged @ property-effects.js:1922
_flushProperties @ properties-changed.js:384
_invalidateProperties @ property-effects.js:1748
Object.defineProperty.set @ properties-changed.js:170
(anonymous) @ vaadin-grid-mixin.js:735
_updateRow @ vaadin-grid-mixin.js:669
(anonymous) @ vaadin-grid-mixin.js:900
iterateChildren @ vaadin-grid-helpers.js:26
_renderColumnTree @ vaadin-grid-mixin.js:899
_columnTreeChanged @ vaadin-grid-mixin.js:842
runMethodEffect @ property-effects.js:1038
runEffects @ property-effects.js:140
_propertiesChanged @ property-effects.js:1922
_flushProperties @ properties-changed.js:384
_invalidateProperties @ property-effects.js:1748
Object.defineProperty.set @ properties-changed.js:170
_updateColumnTree @ vaadin-grid-dynamic-columns-mixin.js:112
(anonymous) @ vaadin-grid-dynamic-columns-mixin.js:94
(anonymous) @ debounce.js:84
microtaskFlush @ async.js:35
(anonymous) @ async.js:179

@yuriy-fix yuriy-fix transferred this issue from vaadin/react-components Jun 27, 2024
@yuriy-fix yuriy-fix transferred this issue from vaadin/hilla Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants