ImmutableContext and ImmutableStructure truncate Instant values to milliseconds

Aberta Para iniciantes
#2,070 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

@RaphaelFakhri já está trabalhando nisso.

Desde 4/11/2026.

  • #2071 de @RaphaelFakhri — aberto

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
76/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
java
Domínio
backend

Direção de pesquisa

Comece lendo Value.clone() e os caminhos de construção e leitura de ImmutableStructure descritos na issue. Verifique ImmutableStructureTest.MutatingGetInstantValueShouldNotChangeOriginalValue, que atualmente trunca a entrada para milissegundos. O trabalho está concluído quando um instante com precisão inferior a um milissegundo é retornado sem alterações por getValue(key).asInstant(), asMap() e asObjectMap().

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

An Instant stored in an ImmutableStructure or ImmutableContext loses its sub-millisecond precision. getValue(key).asInstant(), asMap() and asObjectMap() return an instant truncated to milliseconds, so the value read back is not equal to the value stored.

Value.clone() copies instants with Instant.ofEpochMilli(instant.toEpochMilli()), and ImmutableStructure clones every value on construction and on each read.

Expected behavior

The stored Instant is returned unchanged. Instant is immutable, so a copy is not needed.

Steps to reproduce

Instant instant = Instant.ofEpochSecond(1_700_000_000L, 123_456_789L);
EvaluationContext ctx = new ImmutableContext(Map.of("t", new Value(instant)));

System.out.println(instant);                          // 2023-11-14T22:13:20.123456789Z
System.out.println(ctx.getValue("t").asInstant());    // 2023-11-14T22:13:20.123Z

Instant.now() has microsecond or finer resolution on Java 9 and later, so contexts built from Instant.now() are affected in practice. The existing ImmutableStructureTest.MutatingGetInstantValueShouldNotChangeOriginalValue test calls truncatedTo(ChronoUnit.MILLIS) to avoid this.

Linguagem predominante
Java
Estrelas
129
Forks
62
Merge médio
6h 37min
PRs com merge (30d)
12

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de open-feature/java-sdk

Todas as issues de open-feature/java-sdk

Issues semelhantes

Mais issues de Java

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.