-
Notifications
You must be signed in to change notification settings - Fork 150
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
Get lineTypes as LanguageService.FlushLogLines or getJobLog for rest api #603
Comments
Ok, I now understand what you're asking; took a little research. It's gonna take a little more to see about a prototype, but I'm thinking I may be able to do this. The HTTP Access Method would be the simplest, as it looks straight forward. The IOM AM will take more to see, but it seems doable; gotta try and see (have to rework both Java and python sides of that to even see what I can get. The STDIO/SSH Access Methods have no API so this isn't something that could be supported from that Access Method. My current thought would be adding an option on the submit method to ask for this to be returned instead of the 'normal' LOG that's returned. Would that be sufficient if this all works out? Tom |
Yes. Great! |
So, do you need this in IOM? Or just HTTP? I have this working for HTTP already, as this is what's returned from the server, so it' s pretty easy. For IOM, it's gonna take a while to try to prototype and see if I can even get it to behave similarly. It looks doable, but it's gonna take a lot of work and code changes to prove it out. I've prototyped HTTP and pushed it to a branch called Here's a simple test showing how it works:
Thanks, |
I need the IOM. It will be very appreciated! |
Yes, ok, I'm not surprised. I have a prototype working for that today already. It seems to be working, but it needs more testing for sure. When I get it ready built (just been debugging it so it's not 'clean' yet), I'll push it out to the branch so you can try it. Hopefully before the end of the day today; I have a 4 day weekend. |
OK, I've pushed what I have for IOM to the branch. So, if you can try it out before I get back next Wed., that'd be great. I still have more to do to vet all of this, especially the IOM as it's a lot of code changes. Also, the TYPE in IOM comes back as a number, so I have to try to find a mapping of those to the strings I get back in the HTTP am. For the moment, here's what I get with the same test case as above:
|
HA! I found it! Now I'm good for a long weekend!
|
It helps me. Thanks. |
Sorry, been off on other things, but since this seems to be what you're asking for, and I'm not seeing any issues with it, I 'm good to merge it in and build a new release with it. Does that work for you? Is it still working for you without issue? Thanks, |
It works for me! please go ahead. |
This is now in the current production release, V5.14.0. Go ahead and install that so you have the latest. I'll close this ticket. Let me know if you need anything else! |
Is your feature request related to a problem? Please describe.
The line types (normal, source, error, warning, etc) can help identify issue location in the original code and can help throw usefull error message.
Describe the solution you'd like
Return an array with item like
The text was updated successfully, but these errors were encountered: