You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by GitBox <gi...@apache.org> on 2020/04/02 10:56:37 UTC
[GitHub] [zeppelin] qiguoxiaosheng opened a new pull request #3712: Fix the
ConcurrentModificationException occured when connect presto via JDBC
generic interpreter
qiguoxiaosheng opened a new pull request #3712: Fix the ConcurrentModificationException occured when connect presto via JDBC generic interpreter
URL: https://github.com/apache/zeppelin/pull/3712
### What is this PR for?
This PR is to fix a issue of connect presto via jdbc interpreter when more than one element needs to be removed from the jdbc properties. In this PR, I only change a method which iterate keys from jdbc properties. I tested it manully with presto-0.215.
### What type of PR is it?
[Bug Fix]
### Todos
* [ ] - Task
### What is the Jira issue?
### How should this be tested?
Compiles locally, no unit test performed.
More than one element needs to be removed in the properties.
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [zeppelin] zjffdu commented on issue #3712: [ZEPPELIN-4721]Fix the
ConcurrentModificationException occured when connect presto via JDBC
generic interpreter
Posted by GitBox <gi...@apache.org>.
zjffdu commented on issue #3712: [ZEPPELIN-4721]Fix the ConcurrentModificationException occured when connect presto via JDBC generic interpreter
URL: https://github.com/apache/zeppelin/pull/3712#issuecomment-608263305
LGTM
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [zeppelin] asfgit closed pull request #3712: [ZEPPELIN-4721]Fix the
ConcurrentModificationException occured when connect presto via JDBC
generic interpreter
Posted by GitBox <gi...@apache.org>.
asfgit closed pull request #3712: [ZEPPELIN-4721]Fix the ConcurrentModificationException occured when connect presto via JDBC generic interpreter
URL: https://github.com/apache/zeppelin/pull/3712
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [zeppelin] alexott commented on issue #3712: Fix the
ConcurrentModificationException occured when connect presto via JDBC
generic interpreter
Posted by GitBox <gi...@apache.org>.
alexott commented on issue #3712: Fix the ConcurrentModificationException occured when connect presto via JDBC generic interpreter
URL: https://github.com/apache/zeppelin/pull/3712#issuecomment-607809109
Can you file a JIRA, so we can put its number into changelog?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services