[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Reviewed-on: http://gerrit.cloudera.org:8080/9030 Reviewed-by: Alex RodoniReviewed-by: Kim Jin Chul Reviewed-by: John Russell Tested-by: Impala Public Jenkins --- M docs/topics/impala_hints.xml M docs/topics/impala_insert.xml M docs/topics/impala_upsert.xml 3 files changed, 48 insertions(+), 5 deletions(-) Approvals: Alex Rodoni: Looks good to me, but someone else must approve Kim Jin Chul: Looks good to me, but someone else must approve John Russell: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 3 Gerrit-Owner: Kim Jin Chul Gerrit-Reviewer: Alex Behm Gerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Vuk Ercegovac
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Vuk Ercegovac Gerrit-Comment-Date: Wed, 04 Apr 2018 00:55:56 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: Build started: https://jenkins.impala.io/job/gerrit-docs-submit/222/ -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Vuk Ercegovac Gerrit-Comment-Date: Wed, 04 Apr 2018 00:40:36 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: Could a committer start the jenkins job, please? -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Vuk Ercegovac Gerrit-Comment-Date: Wed, 04 Apr 2018 00:22:47 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: Thank you @Jin Chul! Could someone review the content and give +2? -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Vuk Ercegovac Gerrit-Comment-Date: Wed, 28 Mar 2018 16:12:31 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
Kim Jin Chul has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: Code-Review+1 (1 comment) @Rodoni, sorry for the late. http://gerrit.cloudera.org:8080/#/c/9030/2/docs/topics/impala_upsert.xml File docs/topics/impala_upsert.xml: http://gerrit.cloudera.org:8080/#/c/9030/2/docs/topics/impala_upsert.xml@71 PS2, Line 71: [hint_clause] > Just to confirm: unlike with INSERT, is it correct that the hint for UPSERT No, the optional hints can be placed between UPSERT and INTO. I think the current document is fine to me. -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Vuk Ercegovac Gerrit-Comment-Date: Wed, 28 Mar 2018 11:26:37 + Gerrit-HasComments: Yes
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: Jin Chul, Could you please respond to the latest comment so that we can commit this for the 2.12 release? Thank you! -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Vuk Ercegovac Gerrit-Comment-Date: Tue, 27 Mar 2018 22:45:30 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
John Russell has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/9030/2/docs/topics/impala_upsert.xml File docs/topics/impala_upsert.xml: http://gerrit.cloudera.org:8080/#/c/9030/2/docs/topics/impala_upsert.xml@71 PS2, Line 71: [hint_clause] Just to confirm: unlike with INSERT, is it correct that the hint for UPSERT can _only_ appear ahead of the table ref? -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Vuk Ercegovac Gerrit-Comment-Date: Thu, 22 Mar 2018 21:24:03 + Gerrit-HasComments: Yes
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
John Russell has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: I added Vuk, Dimitris, and Alex as reviewers - the people who did tech review for the code updates, at https://gerrit.cloudera.org/#/c/8676/ -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: Alex Behm Gerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: Dimitris Tsirogiannis Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Reviewer: Vuk Ercegovac Gerrit-Comment-Date: Thu, 22 Mar 2018 21:17:31 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: Could anyone who can review and approve this? -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Comment-Date: Thu, 22 Mar 2018 18:42:32 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: Alex Rodoni Gerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Reviewer: Lars Volker Gerrit-Comment-Date: Tue, 13 Mar 2018 03:48:38 + Gerrit-HasComments: No
[Impala-ASF-CR] IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT
Kim Jin Chul has posted comments on this change. ( http://gerrit.cloudera.org:8080/9030 ) Change subject: IMPALA-4168: [DOCS] Adds Oracle-style hint placement for INSERT/UPSERT .. Patch Set 2: > In addition to the text that was actually added, I'll look in > impala_hints.xml and see if there is info in there that also needs > to change. I added some examples for Oracle-style hint placement for INSERT/UPSERT into impala_hints.xml. I guess some examples would be redundant. They might be overkill. Please give me your advice for my concern. I didn't mention about the new placement in impala_hints because I think this part is to address for hint itself. -- To view, visit http://gerrit.cloudera.org:8080/9030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I43e0a782087c2e67f2e012424fb9261be445efc9 Gerrit-Change-Number: 9030 Gerrit-PatchSet: 2 Gerrit-Owner: Kim Jin ChulGerrit-Reviewer: John Russell Gerrit-Reviewer: Kim Jin Chul Gerrit-Comment-Date: Fri, 26 Jan 2018 01:03:59 + Gerrit-HasComments: No