JanusGraph/janusgraph

hasId does not convert int to Long

Aberta

#1.554 aberto em 6 de mai. de 2019

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)Java (1.137 forks)batch import
area/corehelp wantedkind/enhancement

Métricas do repositório

Stars
 (4.985 estrelas)
Métricas de merge de PR
 (Mesclagem média 3d 18h) (12 fundiu PRs em 30d)

Description

If an int is passed instead of a String or a Long to hasId() it does not get converted and results won't be yielded. This behavior has been report in issue #706, but I think this would be a nice improvement.

I think looking at the HasStepFolder.foldInIds() method might be a good place to start.

Additional info can be found in this stack overflow question and this tinkerpop issue

Guia do colaborador

Direção de pesquisa
Inspecione o método foldInIds em HasStepFolder.java e rastreie como hasId lida com tipos numéricos do adaptador TinkerPop.
Pilha de tecnologia
java
Domain
backenddatabase
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitJava
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60