getappmap / getappmap/appmap-java

Illformed appmaps get generated

Ouverte
#26 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
Java
Étoiles
88
Forks
17
Merge moyen
2 j 2 h
PR mergées (30 j)
1

Description

There are circumstances where the events section of an appmap created by appmap-java is not created correctly.

For example, the appmap for org.elasticsearch.search.aggregations.bucket.terms.BytesKeyedBucketOrdsTests contains the sequences 74},,{" and $"},]}:

Alans-MacBook-Pro org.elasticsearch.search.aggregations$ tail -c+470630560 testExplicitCollectsFromSingleBucket.appmap.json | head -100
tic":false,"thread_id":74},,{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},{"$ref":"$"},]}

The appmap for org.elasticsearch.search.aggregations.metrics.HyperLogLogPlusPlusTests.testMerge contains the sequence ":[,{":

Alans-MacBook-Pro org.elasticsearch.search.aggregations$ tail -c+$(expr 427988011 - 50) testMerge.appmap.json | head -c+200 | less
"lineno":72,"method_id":"readIntLE","parameters":[,{"defined_class":"org.elasticsearch.common.util.AbstractBigArray","event":"return","id":1159599,"lineno":81,"method_id":"size","path":"src/main/java/

The appmap for org.elasticsearch.search.aggregations.metrics.MedianAbsoluteDeviationAggregatorTests contains the sequence main/java,{"defined_class":":

Alans-MacBook-Pro org.elasticsearch.search.aggregations$ tail -c+$(expr 4692538 - 100) testSomeMatchesSortedNumericDocValues.appmap.json | head -c+150
"object_id":719878273,"value":"[Ljava.lang.String;@2ae87881"}],"path":"src/main/java,{"defined_class":"org.elasticsearch.common.settings.AbstractScope

It's possible that there were errors generated during the execution of these tests.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par reproduire la génération d’appmap avec les tests Elasticsearch indiqués, notamment BytesKeyedBucketOrdsTests, HyperLogLogPlusPlusTests.testMerge et MedianAbsoluteDeviationAggregatorTests. Inspectez les sections d’événements générées autour des séquences d’octets signalées et comparez-les à une structure JSON valide. Le travail est terminé lorsque ces tests ne produisent plus de fichiers appmap malformés.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
java
Domaine
backend
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.