Re: Review Request 69550: Add credential store env properties redaction in JobConf

2018-12-11 Thread Vihang Karajgaonkar via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69550/#review211199 --- common/src/java/org/apache/hadoop/hive/conf/HiveConfUtil.java

[GitHub] hive pull request #505: update Readme to fix style issue with Md

2018-12-11 Thread b-slim
GitHub user b-slim opened a pull request: https://github.com/apache/hive/pull/505 update Readme to fix style issue with Md You can merge this pull request into a Git repository by running: $ git pull https://github.com/b-slim/hive patch-1 Alternatively you can review and

Re: Review Request 69550: Add credential store env properties redaction in JobConf

2018-12-11 Thread Denys Kuzmenko via Review Board
> On Dec. 11, 2018, 4:30 p.m., Vihang Karajgaonkar wrote: > > common/src/java/org/apache/hadoop/hive/conf/HiveConfUtil.java > > Lines 217 (patched) > > > > > > Do we need to do redact for the Spark config as well?

Re: [DISCUSS] Consistent Timestamps across Hadoop

2018-12-11 Thread Zoltan Ivanfi
Hi Owen, Thanks, I think your email contains a great summary of the problems tackled in the proposal. I would like highlight two particular topics from the discussion that we are having in the comments (details can be read in the document

Re: [NOTICE] Mandatory relocation of Apache git repositories on git-wip-us.apache.org

2018-12-11 Thread Slim Bouguerra
+1 on Early move. Having direct write access to Github is a good first step to move toward one place that can be used to review and merge pull requests IMO. On Dec 11, 2018, at 12:06 PM, Alan Gates wrote: This includes the hive project, as we're on the older git-wip-us. Do we want to move

Re: Review Request 69462: HIVE-20936

2018-12-11 Thread Jaume Marhuenda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69462/ --- (Updated Dec. 11, 2018, 11:45 p.m.) Review request for hive. Repository:

[GitHub] hive pull request #505: update Readme to fix style issue with Md

2018-12-11 Thread b-slim
Github user b-slim closed the pull request at: https://github.com/apache/hive/pull/505 ---

[jira] [Created] (HIVE-21031) Array with one empty string is inserted as an empty array

2018-12-11 Thread Patrick Byrnes (JIRA)
Patrick Byrnes created HIVE-21031: - Summary: Array with one empty string is inserted as an empty array Key: HIVE-21031 URL: https://issues.apache.org/jira/browse/HIVE-21031 Project: Hive

Re: Review Request 69550: Add credential store env properties redaction in JobConf

2018-12-11 Thread Vihang Karajgaonkar via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69550/#review211205 --- Ship it! Ship It! - Vihang Karajgaonkar On Dec. 11, 2018,

Fwd: [NOTICE] Mandatory relocation of Apache git repositories on git-wip-us.apache.org

2018-12-11 Thread Alan Gates
This includes the hive project, as we're on the older git-wip-us. Do we want to move sooner or later? Alan. -- Forwarded message - From: Daniel Gruno Date: Fri, Dec 7, 2018 at 8:52 AM Subject: [NOTICE] Mandatory relocation of Apache git repositories on git-wip-us.apache.org To:

Re: [NOTICE] Mandatory relocation of Apache git repositories on git-wip-us.apache.org

2018-12-11 Thread Slim Bouguerra
Then next step will be to formalize/define the process within the community thought. On Tue, Dec 11, 2018 at 12:55 PM Alan Gates wrote: > IIUC it's the only step necessary to have one place where we could review > and merge pull requests. There'd be no more need for attaching patches to >

Re: Review Request 69550: Add credential store env properties redaction in JobConf

2018-12-11 Thread Vihang Karajgaonkar via Review Board
> On Dec. 11, 2018, 4:30 p.m., Vihang Karajgaonkar wrote: > > common/src/java/org/apache/hadoop/hive/conf/HiveConfUtil.java > > Lines 217 (patched) > > > > > > Do we need to do redact for the Spark config as well? >

Re: [NOTICE] Mandatory relocation of Apache git repositories on git-wip-us.apache.org

2018-12-11 Thread Vineet Garg
+1 on early move. > On Dec 11, 2018, at 12:46 PM, Slim Bouguerra wrote: > > +1 on Early move. > > Having direct write access to Github is a good first step to move toward > one place that can be used to review and merge pull requests IMO. > > On Dec 11, 2018, at 12:06 PM, Alan Gates wrote: >

Re: [NOTICE] Mandatory relocation of Apache git repositories on git-wip-us.apache.org

2018-12-11 Thread Alan Gates
IIUC it's the only step necessary to have one place where we could review and merge pull requests. There'd be no more need for attaching patches to JIRA. Of course we'd need to update our CI processes to match. Alan. On Tue, Dec 11, 2018 at 12:46 PM Slim Bouguerra wrote: > +1 on Early move.

[GitHub] hive pull request #506: HIVE-20911: External Table Replication for Hive

2018-12-11 Thread anishek
GitHub user anishek opened a pull request: https://github.com/apache/hive/pull/506 HIVE-20911: External Table Replication for Hive You can merge this pull request into a Git repository by running: $ git pull https://github.com/anishek/hive HIVE-20911 Alternatively you can

[jira] [Created] (HIVE-21030) Add credential store env properties redaction in JobConf

2018-12-11 Thread Denys Kuzmenko (JIRA)
Denys Kuzmenko created HIVE-21030: - Summary: Add credential store env properties redaction in JobConf Key: HIVE-21030 URL: https://issues.apache.org/jira/browse/HIVE-21030 Project: Hive

Review Request 69550: Add credential store env properties redaction in JobConf

2018-12-11 Thread Denys Kuzmenko via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69550/ --- Review request for hive, Peter Vary and Vihang Karajgaonkar. Repository:

Re: Review Request 69550: Add credential store env properties redaction in JobConf

2018-12-11 Thread Denys Kuzmenko via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69550/ --- (Updated Dec. 11, 2018, 12:43 p.m.) Review request for hive, Peter Vary and