Skip to content

Commit c20e11c

Browse files
MopsgeschwindigkeitAlexander ObereggeraobereggMarco Julian Moser
authored
resolving wrong reference (#438)
* add static partition assignment possibilities * add global partitions as actives as well * pin mode as long we didn't switch to newest faust version that supports python 3.11 * first version of global global tables * add GlobalGlobalTable Object * have global global as attribute of global table * use use_partitioner also for having all partitions as standby * Update requirements.txt * fix tests, assert that global flag is set in case global global is used, linting * remove test artefact * fix linting again * rename global global to synchronize_all_active_partitions * fix linting * add base.py changes * fixed synchornized_all_active_partitions attribute handling in GlobalTable Constructor * resolved reference error self.table.use_partitioner -> table.usepartitioner * run code formatter Co-authored-by: Alexander Oberegger <[email protected]> Co-authored-by: aoberegg <[email protected]> Co-authored-by: Marco Julian Moser <[email protected]>
1 parent aaadab2 commit c20e11c

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

faust/assignor/partition_assignor.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -336,10 +336,7 @@ def _global_table_standby_assignments(
336336
# the table and consumer group B, C, D only consuming. With the
337337
# synchronize_all_active_partitions flag it's possible to have
338338
# shared state over multiple consumer groups.
339-
if (
340-
table.synchronize_all_active_partitions
341-
or self.table.use_partitioner
342-
):
339+
if table.synchronize_all_active_partitions or table.use_partitioner:
343340
standby_partitions = all_partitions
344341
else: # Only add those partitions as standby which aren't active
345342
standby_partitions = all_partitions - active_partitions

0 commit comments

Comments
 (0)