Skip to content

Conversation

@viirya
Copy link
Member

@viirya viirya commented Nov 19, 2015

JIRA: https://issues.apache.org/jira/browse/SPARK-11817

Instead of return None, we should truncate the fractional seconds to prevent inserting NULL.

@SparkQA
Copy link

SparkQA commented Nov 19, 2015

Test build #46310 has finished for PR 9834 at commit 154f831.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):\n * class LogisticRegressionModelWriter(instance: LogisticRegressionModel)\n

@yhuai
Copy link
Contributor

yhuai commented Nov 20, 2015

Thank you! LGTM. I am merging it to master, branch 1.6 and branch 1.5.

asfgit pushed a commit that referenced this pull request Nov 20, 2015
…ting a NULL

JIRA: https://issues.apache.org/jira/browse/SPARK-11817

Instead of return None, we should truncate the fractional seconds to prevent inserting NULL.

Author: Liang-Chi Hsieh <viirya@appier.com>

Closes #9834 from viirya/truncate-fractional-sec.

(cherry picked from commit 60bfb11)
Signed-off-by: Yin Huai <yhuai@databricks.com>
asfgit pushed a commit that referenced this pull request Nov 20, 2015
…ting a NULL

JIRA: https://issues.apache.org/jira/browse/SPARK-11817

Instead of return None, we should truncate the fractional seconds to prevent inserting NULL.

Author: Liang-Chi Hsieh <viirya@appier.com>

Closes #9834 from viirya/truncate-fractional-sec.

(cherry picked from commit 60bfb11)
Signed-off-by: Yin Huai <yhuai@databricks.com>
@asfgit asfgit closed this in 60bfb11 Nov 20, 2015
@viirya viirya deleted the truncate-fractional-sec branch December 27, 2023 18:18
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.

3 participants