Github user huaxingao commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-214437787
@HyukjinKwon
I will continue working on this and finish the work this week.
---
If your project is set up for it, you can reply to this email and have your
Github user HyukjinKwon commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-214143879
@huaxingao Do you mind If I submit a PR based on this if you are not
working on this? (It looks it has been inactive for few months!)
---
If your project is set
Github user huaxingao commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-170701187
@yhuai
Sorry for the late reply. I waited for my coworker Luciano to come back
from vacation today to check with him about his DB2 docker test status. He has
Github user huaxingao commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-166033287
@yhuai
JDBCSuite uses H2 database. It seems that for char(n) data type, either H2
database doesn't pad, or the H2 JDBC driver already trims the empty space for
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-166058803
I guess we can try to add a docker test. Can you try to add one in
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-165297054
Can you add a test?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user huaxingao commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-164680145
@andrewor14
Thanks a lot for your comment. I will change to what you suggested.
In the same method, case DecimalConversion has the similar code. Shall I
Github user 3ourroom commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-164680307
NAVER - http://www.naver.com/
3ourr...@naver.com ëê» ë³´ë´ì ë©ì¼
ì´ ë¤ìê³¼ ê°ì ì´ì ë¡
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-164945607
Jenkins, ok to test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-164946337
**[Test build #47763 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47763/consoleFull)**
for PR 10262 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-164966501
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-164966495
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-164966108
**[Test build #47763 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47763/consoleFull)**
for PR 10262 at commit
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-164583283
@yhuai
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/10262#discussion_r47570429
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRDD.scala
---
@@ -422,7 +422,14 @@ private[sql] class JDBCRDD(
GitHub user huaxingao opened a pull request:
https://github.com/apache/spark/pull/10262
[SPARK-12270][SQL]remove empty space after getString from database
{code}
conn.prepareStatement(
"create table people (name char(32)").executeUpdate()
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10262#issuecomment-163812129
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
17 matches
Mail list logo