Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port to the latest Javascript driver version #666

Closed
vmarkovtsev opened this issue Mar 6, 2019 · 8 comments
Closed

Port to the latest Javascript driver version #666

vmarkovtsev opened this issue Mar 6, 2019 · 8 comments
Assignees
Labels
blocked Blocked task or issue format Issues related to format analyzer medium Medium size

Comments

@vmarkovtsev
Copy link
Collaborator

We need to investigate why the newer releases do not work with our code, report possible bugs to Babelfish and switch to the latest JS driver version.

@vmarkovtsev
Copy link
Collaborator Author

BTW @zurk this can be fused with the annotations refactoring.

@zurk
Copy link
Contributor

zurk commented Mar 6, 2019

I'd like to do it separately.

@zurk zurk added the format Issues related to format analyzer label Mar 6, 2019
@vmarkovtsev
Copy link
Collaborator Author

"The latest JS driver" does not mean UASTv2 - it means upgrading the driver. And regarding the annotations, I am afraid that you will have to repeat some of the work twice otherwise.

@vmarkovtsev
Copy link
Collaborator Author

If we see that the latest JS driver works incorrectly with UASTv1 protocol, we should report it as an issue and ask LA to fix it ASAP.

@zurk
Copy link
Contributor

zurk commented Mar 6, 2019

If we see that the latest JS driver works incorrectly with UASTv1 protocol, we should report it as an issue and ask LA to fix it ASAP.

Ok, I know that there are some issues for sure. AFAIK, UASTv1 is not clearly the same as an annotated mode in new drivers. Will see.

And regarding the annotations, I am afraid that you will have to repeat some of the work twice otherwise.

It is fine. Annotation is about the shape of the code and this issue can touch its logic.

@zurk
Copy link
Contributor

zurk commented Mar 11, 2019

I report several minor issues related to javascript driver:

However, the biggest problem is on our side. We currently use v1.2.0 driver and the driver wrongly returns Node positions for Unicode string. v2.6.0 returns offsets for byte string, so we should recalculate them back to Unicode.

@zurk
Copy link
Contributor

zurk commented Mar 12, 2019

One more. bblfsh/javascript-driver#67
I think that there are more files that are parsable with v1.2.0 but not with v2.6.0
I am going to create a list tomorrow.

@zurk
Copy link
Contributor

zurk commented Mar 13, 2019

PR: #692
Issue is blocked until v.2.7.0 driver release.

@zurk zurk added blocked Blocked task or issue medium Medium size labels Mar 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Blocked task or issue format Issues related to format analyzer medium Medium size
Projects
None yet
Development

No branches or pull requests

2 participants