Comments (11)
Oooooh yes, I totally forgot about that, I also lost many hours with that the other day with that cache...
from sdk.
Fixed by PR #135 when merged.
from sdk.
Wow! So awesome, did not realize #135 is merged - going to test now.
from sdk.
I have tested the the latest bblfsh server
\w #135 merged and I still get buffer size exceeded
sometimes :/ not sure if that is related to this particular buffer size..
Here is the 12k LOC file it happened on
https://github.com/bit-man/bit-tools/blob/master/GenUnitTests-ECI2012/src/test/java/ej2/conInvariante/RandoopTestConInvariante1.java
I would be happy to just skip such files, but after this error - server hangs forever
Might be related to bblfsh/bblfshd#42
from sdk.
Very odd since that message is not in the SDK code anymore, I'll try with the docker image (but that SDK code is really used in the drivers which also have new versions compiled against the latest SDK).
from sdk.
I can't reproduce it. Steps I took (using the workaround posted on Slack for the problem with the current server and env variables):
# Remove old images
docker rm (docker ps -a -q)
docker rmi --force (docker images -q)
# workaround
export BBLFSH_DRIVER_IMAGES="a=a"
# Run the server
docker run -e BBLFSH_DRIVER_IMAGES --privileged -p 9432:9432 --name bblfsh bblfsh/server
# Send the file
docker run -v $(pwd):/work --link bblfsh bblfsh/server bblfsh client --address=bblfsh:9432 /work/shit.java
time="2017-07-05T19:27:06Z" level=debug msg="reading file: /work/shit.java"
time="2017-07-05T19:27:06Z" level=debug msg="dialing server at bblfsh:9432"
time="2017-07-05T19:27:06Z" level=debug msg="instantiating service client"
time="2017-07-05T19:27:06Z" level=debug msg="sending request"
(super huge UAST is printed)
I tried again with several consecutive requests just in case it was hanging at the second request like in server bug #36 but it works perfectly.
from sdk.
Ok, I was missing --transport=docker-daemon
when running server, so I guess drivers were fetched from DockerHub instead of using latest build/taggeg :\
But this does not explain why buffer size exceeded
show up, does it?
from sdk.
Actually, I should be using the version from Dockerhub to test it, let me test again without that.
from sdk.
I didn't use --transport=docker-daemon. I just tried without the workaround (since @abeaumont fixed that and published the new image) and it worked again without problems:
docker rm (docker ps -a -q)
docker rmi --force (docker images -q)
docker run --privileged -p 9432:9432 --name bblfsh bblfsh/server
docker run -v (pwd):/work --link bblfsh bblfsh/server bblfsh client --address=bblfsh:9432 /work/shit.java
from sdk.
Closing since the fix was merged.
from sdk.
Ok, after more digging - the reason I was getting this is that if you do not use transport - by default all the runtime, that includes driver containers, is cached under ./tmp
and is never updated.
AFAIK there are plans to add update
command eventually to bblfsh that would help to avoid it, but that's in future.
I'm removing it manually and it should work now.
from sdk.
Related Issues (20)
- drivers output on debug level does not include requests
- driver failure: native driver is not running HOT 6
- schema: Add string interpolation UAST type HOT 1
- Grpc: received message larger than max (5797446 vs. 4194304) HOT 1
- Clarify backward compatibility rules for uast packages
- Parent covers children invariant is not respected HOT 2
- Parent covers children invariant is not respected HOT 7
- uast.TokenOf returns unexpected results for position nodes
- Support persistent UAST hashes HOT 1
- Strange requests during driver discovery tests HOT 1
- Incorrect check for invalid positions HOT 1
- Move protocol to a separate package
- ChildrenOrder unexpected behaviour HOT 2
- .filter() - Invalid memory address or nil pointer dereference HOT 5
- Feature request: UAST diffing HOT 6
- Fix xpath library wrong query failures in case we continue supporting it
- Update drivers template with new performance tests
- Roles are sorted in encoding step but not in decoding one
- Comments including utf8 characters are not parsed correctly HOT 8
- Automation: driver releases
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from sdk.