Fetching structs containing datetimes fails silently
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- sql, typescript
- Domaine
- databases
Piste de recherche
Commencez par lib/result/JsonResult.ts aux lignes référencées 89 et 121, puis reproduisez l’échec avec la struct fournie contenant des valeurs datetime pour created_at et updated_at. Vérifiez comment l’analyse se rabat sur defaultValue, et définissez comme terminé le fait de gérer cette entrée sans perte silencieuse de données, tout en déterminant si un avertissement doit être journalisé.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
If the data being parsed contains datetimes, it fails and returns defaultValue. Example error and data are below:
SyntaxError: Unexpected number in JSON at position 39
at JSON.parse (<anonymous>)
at JsonResult.toJSON (./databricks-sql-nodejs/dist/result/JsonResult.js:93:25)
at JsonResult.convertData (./databricks-sql-nodejs/dist/result/JsonResult.js:64:29)
at ./databricks-sql-nodejs/dist/result/JsonResult.js:47:25
at Array.map (<anonymous>)
at JsonResult.getSchemaValues (./databricks-sql-nodejs/dist/result/JsonResult.js:43:35)
at ./databricks-sql-nodejs/dist/result/JsonResult.js:27:62
at Array.reduce (<anonymous>)
at JsonResult.getRows (./databricks-sql-nodejs/dist/result/JsonResult.js:27:28)
at ./databricks-sql-nodejs/dist/result/JsonResult.js:16:31
The string that's failing to parse is below (partially scrubbed)
'{"id":414247,"created_at":2021-12-21 21:33:59.339,"updated_at":2021-12-21 21:33:59.339,"deleted_at":null,"s3_bucket":"thebucket","s3_key":"c411f24d-1b4a-4eb0-b25b-d2287c7ba3c0"}'
Also, would it make sense to at least log a warning if parsing fails and returns the default value?
- Langage dominant
- TypeScript
- Étoiles
- 37
- Forks
- 51
- Merge moyen
- 5 h 56 min
- PR mergées (30 j)
- 9
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de databricks/databricks-sql-nodejs
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
databricks/databricks-sql-nodejs#526 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
databricks/databricks-sql-nodejs#503 ·
Les mainteneurs répondent en général sous 1 jour
-
Docs folder deleted in 1.8.4Ouverteengineer-bot
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
databricks/databricks-sql-nodejs#274 · 1 commentaire · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
databricks/databricks-sql-nodejs#525 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
databricks/databricks-sql-nodejs#524 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de databricks/databricks-sql-nodejs
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
external-issue to-triage
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
diegosouzapw/OmniRoute#15401 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
code-yeongyu/oh-my-openagent#9454 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
smart-village-solutions/sva-studio#1654 ·
Les mainteneurs répondent en général sous 1 jour