Refactoring of the ADM parser

shamim8888 / asterixdb

Automatically exported from code.google.com/p/asterixdb

0 stars 0 forks source link

Refactoring of the ADM parser #663

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago

I would propose a refactoring of the ADM parser that currently shows duplicated 
code, huge classes (too many lines of code), and usability problems with the 
lexer generator.

This would also:
- allow a cleaner design for the parser
- increase the easiness of introducing new adm data-types
- improve the maintainability
- remove the need of the complex maven configuration for the lexer-generator

Original issue reported on code.google.com by diegogio...@gmail.com on 30 Oct 2013 at 10:48

GoogleCodeExporter commented 9 years ago

Hi Diego,

is there still hope that would will get to this?

Original comment by westm...@gmail.com on 30 Jul 2014 at 7:03

GoogleCodeExporter commented 9 years ago

the work was almost ready and uploaded in the branch 
diego/feature/adm_parser_v3  ( 
https://code.google.com/p/asterixdb/source/browse/?name=diego%2Ffeature%2Fadm_pa
rser_v3) but I never merged because I found a performance regression that I 
could not understand the origin.

After that I didn't follow the progress of the codebase, so I guess this issue 
is obsolete now.

Original comment by diegogio...@gmail.com on 28 Mar 2015 at 6:42