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

Update Java worker config #550

Merged
merged 1 commit into from
Jul 6, 2018
Merged

Update Java worker config #550

merged 1 commit into from
Jul 6, 2018

Conversation

pragnagopa
Copy link
Member

@pragnagopa pragnagopa commented Jun 22, 2018

PR Azure/azure-functions-host#3047 moves JavaWorker to use generic worker provider.

@pragnagopa pragnagopa requested a review from ahmelsayed June 22, 2018 21:02
@pragnagopa
Copy link
Member Author

pragnagopa commented Jun 22, 2018

@asavaritayal - please follow up with VSCode as setting debug port for java functions needs to be updated once this PR is merged in

-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address={portnumber} 

@pragnagopa
Copy link
Member Author

/cc @nturinski @fiveisprime

@pragnagopa
Copy link
Member Author

@alrod please merge this when functions host is ready for release

@pragnagopa pragnagopa changed the title [Do not merge]Update Java worker config Update Java worker config Jul 6, 2018
@pragnagopa
Copy link
Member Author

@ahmelsayed - Please review.

@ahmelsayed
Copy link
Contributor

@pragnagopa regarding this issue Azure/azure-functions-host#3080. Do you think we could get a fix in this release?

@ahmelsayed ahmelsayed merged commit ff5683e into Azure:master Jul 6, 2018
@pragnagopa
Copy link
Member Author

Working on the fix for : Azure/azure-functions-host#3080

@pragnagopa
Copy link
Member Author

@jdneo - You need to pass -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address={portnumber} to CLI to set debug port on the Java Worker

@pragnagopa
Copy link
Member Author

@nturinski / @fiveisprime/ @asavaritayal - FYI, the functions runtime and CLI release with these changes is in progress

@pragnagopa
Copy link
Member Author

@EricJizbaMSFT - This is the change for updated debug options for Java

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants