[jira] [Commented] (PHOENIX-6974) Move PBaseColumn to test

2023-06-09 Thread Istvan Toth (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731157#comment-17731157 ] Istvan Toth commented on PHOENIX-6974: -- I'm fine with deleting them. > Move PBaseColumn to test >

[jira] [Commented] (PHOENIX-6973) Add option to CREATE TABLE to skip verification of HBase table

2023-06-09 Thread Viraj Jasani (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6973?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731139#comment-17731139 ] Viraj Jasani commented on PHOENIX-6973: --- [~payert] provided you with the contributor access and

[jira] [Commented] (PHOENIX-6973) Add option to CREATE TABLE to skip verification of HBase table

2023-06-09 Thread Viraj Jasani (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6973?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731051#comment-17731051 ] Viraj Jasani commented on PHOENIX-6973: --- apologies for the confusion [~payert] [~stoty] > Add

[jira] [Comment Edited] (PHOENIX-6974) Move PBaseColumn to test

2023-06-09 Thread Jing Yu (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731048#comment-17731048 ] Jing Yu edited comment on PHOENIX-6974 at 6/9/23 5:51 PM: -- [~stoty] , some of

[jira] [Commented] (PHOENIX-6974) Move PBaseColumn to test

2023-06-09 Thread Jing Yu (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731048#comment-17731048 ] Jing Yu commented on PHOENIX-6974: -- Some of its descendants are never used (even by tests). Should we

[jira] [Commented] (PHOENIX-6962) Write javadoc for different mutation plans within DeleteCompiler.

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731047#comment-17731047 ] ASF GitHub Bot commented on PHOENIX-6962: - virajjasani merged PR #1619: URL:

[GitHub] [phoenix] virajjasani merged pull request #1619: PHOENIX-6962 Write javadoc for different mutation plans within Delete…

2023-06-09 Thread via GitHub
virajjasani merged PR #1619: URL: https://github.com/apache/phoenix/pull/1619 -- 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. To unsubscribe, e-mail:

[jira] [Commented] (PHOENIX-6962) Write javadoc for different mutation plans within DeleteCompiler.

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731045#comment-17731045 ] ASF GitHub Bot commented on PHOENIX-6962: - virajjasani commented on PR #1619: URL:

[GitHub] [phoenix] virajjasani commented on pull request #1619: PHOENIX-6962 Write javadoc for different mutation plans within Delete…

2023-06-09 Thread via GitHub
virajjasani commented on PR #1619: URL: https://github.com/apache/phoenix/pull/1619#issuecomment-1584936776 @shahrs87 anything from you sir before closing this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[jira] [Commented] (PHOENIX-6962) Write javadoc for different mutation plans within DeleteCompiler.

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731044#comment-17731044 ] ASF GitHub Bot commented on PHOENIX-6962: - virajjasani commented on PR #1619: URL:

[GitHub] [phoenix] virajjasani commented on pull request #1619: PHOENIX-6962 Write javadoc for different mutation plans within Delete…

2023-06-09 Thread via GitHub
virajjasani commented on PR #1619: URL: https://github.com/apache/phoenix/pull/1619#issuecomment-1584936377 Thank you @jinggou, +1 -- 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

[GitHub] [phoenix] tkhurana commented on pull request #1619: PHOENIX-6962 Write javadoc for different mutation plans within Delete…

2023-06-09 Thread via GitHub
tkhurana commented on PR #1619: URL: https://github.com/apache/phoenix/pull/1619#issuecomment-1584929401 Thanks @jinggou -- 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.

[jira] [Commented] (PHOENIX-6962) Write javadoc for different mutation plans within DeleteCompiler.

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731043#comment-17731043 ] ASF GitHub Bot commented on PHOENIX-6962: - tkhurana commented on PR #1619: URL:

[GitHub] [phoenix] virajjasani commented on a diff in pull request #1619: PHOENIX-6962 Write javadoc for different mutation plans within Delete…

2023-06-09 Thread via GitHub
virajjasani commented on code in PR #1619: URL: https://github.com/apache/phoenix/pull/1619#discussion_r1224545944 ## phoenix-core/src/main/java/org/apache/phoenix/compile/DeleteCompiler.java: ## @@ -354,7 +354,12 @@ private List getClientSideMaintainedIndexes(TableRef

[jira] [Commented] (PHOENIX-6962) Write javadoc for different mutation plans within DeleteCompiler.

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731032#comment-17731032 ] ASF GitHub Bot commented on PHOENIX-6962: - virajjasani commented on code in PR #1619: URL:

[jira] [Commented] (PHOENIX-6962) Write javadoc for different mutation plans within DeleteCompiler.

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731033#comment-17731033 ] ASF GitHub Bot commented on PHOENIX-6962: - tkhurana commented on code in PR #1619: URL:

[GitHub] [phoenix] tkhurana commented on a diff in pull request #1619: PHOENIX-6962 Write javadoc for different mutation plans within Delete…

2023-06-09 Thread via GitHub
tkhurana commented on code in PR #1619: URL: https://github.com/apache/phoenix/pull/1619#discussion_r1224546560 ## phoenix-core/src/main/java/org/apache/phoenix/compile/DeleteCompiler.java: ## @@ -872,6 +887,12 @@ public QueryPlan getQueryPlan() { } } +/** +

[jira] [Commented] (PHOENIX-6962) Write javadoc for different mutation plans within DeleteCompiler.

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731031#comment-17731031 ] ASF GitHub Bot commented on PHOENIX-6962: - virajjasani commented on code in PR #1619: URL:

[GitHub] [phoenix] virajjasani commented on a diff in pull request #1619: PHOENIX-6962 Write javadoc for different mutation plans within Delete…

2023-06-09 Thread via GitHub
virajjasani commented on code in PR #1619: URL: https://github.com/apache/phoenix/pull/1619#discussion_r1224545944 ## phoenix-core/src/main/java/org/apache/phoenix/compile/DeleteCompiler.java: ## @@ -354,7 +354,12 @@ private List getClientSideMaintainedIndexes(TableRef

[jira] [Commented] (PHOENIX-6962) Write javadoc for different mutation plans within DeleteCompiler.

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731030#comment-17731030 ] ASF GitHub Bot commented on PHOENIX-6962: - tkhurana commented on code in PR #1619: URL:

[GitHub] [phoenix] tkhurana commented on a diff in pull request #1619: PHOENIX-6962 Write javadoc for different mutation plans within Delete…

2023-06-09 Thread via GitHub
tkhurana commented on code in PR #1619: URL: https://github.com/apache/phoenix/pull/1619#discussion_r1224545078 ## phoenix-core/src/main/java/org/apache/phoenix/compile/DeleteCompiler.java: ## @@ -872,6 +887,12 @@ public QueryPlan getQueryPlan() { } } +/** +

[GitHub] [phoenix] tkhurana commented on a diff in pull request #1619: PHOENIX-6962 Write javadoc for different mutation plans within Delete…

2023-06-09 Thread via GitHub
tkhurana commented on code in PR #1619: URL: https://github.com/apache/phoenix/pull/1619#discussion_r1224542740 ## phoenix-core/src/main/java/org/apache/phoenix/compile/DeleteCompiler.java: ## @@ -872,6 +887,12 @@ public QueryPlan getQueryPlan() { } } +/** +

[jira] [Commented] (PHOENIX-6962) Write javadoc for different mutation plans within DeleteCompiler.

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731027#comment-17731027 ] ASF GitHub Bot commented on PHOENIX-6962: - tkhurana commented on code in PR #1619: URL:

[jira] [Commented] (PHOENIX-6962) Write javadoc for different mutation plans within DeleteCompiler.

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6962?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17731025#comment-17731025 ] ASF GitHub Bot commented on PHOENIX-6962: - tkhurana commented on code in PR #1619: URL:

[GitHub] [phoenix] tkhurana commented on a diff in pull request #1619: PHOENIX-6962 Write javadoc for different mutation plans within Delete…

2023-06-09 Thread via GitHub
tkhurana commented on code in PR #1619: URL: https://github.com/apache/phoenix/pull/1619#discussion_r1224539883 ## phoenix-core/src/main/java/org/apache/phoenix/compile/DeleteCompiler.java: ## @@ -354,7 +354,12 @@ private List getClientSideMaintainedIndexes(TableRef tableRef)

[jira] [Commented] (PHOENIX-6910) Scans created during query compilation and execution against salted tables need to be more resilient

2023-06-09 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/PHOENIX-6910?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17730859#comment-17730859 ] ASF GitHub Bot commented on PHOENIX-6910: - stoty commented on PR #1607: URL:

[GitHub] [phoenix] stoty commented on pull request #1607: PHOENIX-6910 Scans created during query compilation and execution aga…

2023-06-09 Thread via GitHub
stoty commented on PR #1607: URL: https://github.com/apache/phoenix/pull/1607#issuecomment-1584151076 I have attached local-verify-run.gz to the ticket. It only fails on the known flakey permission ITs. (It seems that we have tests that tend to fail on slow machines, and also ones