partition_unique leaks metadata references for completed and duplicate records
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 25/100
Direção de pesquisa
Start at partition_unique and inspect the shared retain/release helpers, then add regressions for keep="first", keep="last", and metadata shape. Verify discarded references are released immediately, synchronous completion leaves no references retained, and buffered downstream nodes keep references until processing finishes. An open pull request (#502) is already linked to this issue.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
partition_unique retains metadata references for every input, but does not release them when a duplicate is discarded (keep="first") or replaced (keep="last"). It also emits nested metadata lists, while the shared retain/release helpers expect a flat list of dictionaries. Even a completed partition with a synchronous sink leaves its references retained.
Reproduced on master 5267116f49fc2257e39b44abd8d1e5c5460715f8, Windows Python 3.12.14:
from streamz import Stream, RefCounter
for keep in ("first", "last"):
source = Stream()
output = source.partition_unique(2, keep=keep).sink_to_list()
refs = [RefCounter() for _ in range(3)]
for value, ref in zip((1, 1, 2), refs):
source.emit(value, metadata=[{"ref": ref}])
print(keep, output, [ref.count for ref in refs])
Actual: both modes output [(1, 2)] with counts [1, 1, 1]. Expected: [0, 0, 0] after the synchronous sink finishes; discarded records should release their references immediately. Buffered downstream nodes must still retain references until they finish processing.
I plan a narrow fix in partition_unique: release discarded metadata, flatten retained metadata for emission, and add first/last ownership and metadata-shape regressions. Searches found the original feature PR #389 but no current fix for this issue. Is there existing work or a preferred approach?
AI disclosure: this report and proposed contribution were prepared and reproduced by Codex under the account owner authorization. I am not claiming an active production use case or manual human review.
- Linguagem predominante
- Python
- Estrelas
- 1.3k
- Forks
- 151
- Merge médio
- 17h 39min
- PRs com merge (30d)
- 1
Preparar o ambiente
- Inclui um Dockerfile ou arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de python-streamz/streamz
-
pkg_resources warningAberta
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 35/100
python-streamz/streamz#481 · 4 comentários ·
-
Combining the streamz.Stream.filenames() and streamz.Stream.from_textfile() using dask scatter?Aberta
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
python-streamz/streamz#480 · 2 comentários ·
-
Compile the code into c++Aberta
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 20/100
python-streamz/streamz#479 · 1 comentário ·
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 20/100
python-streamz/streamz#478 · 6 comentários ·
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 10/100
python-streamz/streamz#476 · 17 comentários · 2 reações ·
Todas as issues de python-streamz/streamz
Issues semelhantes
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 72/100
-
[Bug]: Importing weave changes the process-wide localeTalvez já em andamento @himanshu748 assumiu hoje. Abertabug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
-
UX
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 72/100
mediajunkie/piper-morgan-product#2008 · 2 comentários ·