You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by "yihua (via GitHub)" <gi...@apache.org> on 2024/11/04 20:04:39 UTC

[PR] [HUDI-8430] Remove redundant configs in TestInsertTable [hudi]

yihua opened a new pull request, #12202:
URL: https://github.com/apache/hudi/pull/12202

   ### Change Logs
   
   The record merge implementation class of `HoodieSparkValidateDuplicateKeyRecordMerger` and `recordMergeStrategyId` should not be required for Spark SQL INSERT INTO statement for strict insert mode and deduplication after #11943.  This PR removes such configs in `TestInsertTable`.
   
   ### Impact
   
   Makes test realistic based on the expected user experience.
   
   ### Risk level
   
   none
   
   ### Documentation Update
   
   none
   
   ### Contributor's checklist
   
   - [ ] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [ ] Change Logs and Impact were stated clearly
   - [ ] Adequate tests were added if applicable
   - [ ] CI passed
   


-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [HUDI-8430] Remove redundant configs in TestInsertTable [hudi]

Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #12202:
URL: https://github.com/apache/hudi/pull/12202#issuecomment-2455602080

   <!--
   Meta data
   {
     "version" : 1,
     "metaDataEntries" : [ {
       "hash" : "557fd2761e0817b6be38ecd5a5eb7a1a52f2bb46",
       "status" : "PENDING",
       "url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=1491",
       "triggerID" : "557fd2761e0817b6be38ecd5a5eb7a1a52f2bb46",
       "triggerType" : "PUSH"
     } ]
   }-->
   ## CI report:
   
   * 557fd2761e0817b6be38ecd5a5eb7a1a52f2bb46 Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=1491) 
   
   <details>
   <summary>Bot commands</summary>
     @hudi-bot supports the following commands:
   
    - `@hudi-bot run azure` re-run the last Azure build
   </details>


-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [HUDI-8430] Remove redundant configs in TestInsertTable [hudi]

Posted by "wombatu-kun (via GitHub)" <gi...@apache.org>.
wombatu-kun merged PR #12202:
URL: https://github.com/apache/hudi/pull/12202


-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [HUDI-8430] Remove redundant configs in TestInsertTable [hudi]

Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #12202:
URL: https://github.com/apache/hudi/pull/12202#issuecomment-2455599201

   <!--
   Meta data
   {
     "version" : 1,
     "metaDataEntries" : [ {
       "hash" : "557fd2761e0817b6be38ecd5a5eb7a1a52f2bb46",
       "status" : "UNKNOWN",
       "url" : "TBD",
       "triggerID" : "557fd2761e0817b6be38ecd5a5eb7a1a52f2bb46",
       "triggerType" : "PUSH"
     } ]
   }-->
   ## CI report:
   
   * 557fd2761e0817b6be38ecd5a5eb7a1a52f2bb46 UNKNOWN
   
   <details>
   <summary>Bot commands</summary>
     @hudi-bot supports the following commands:
   
    - `@hudi-bot run azure` re-run the last Azure build
   </details>


-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [HUDI-8430] Remove redundant configs in TestInsertTable [hudi]

Posted by "hudi-bot (via GitHub)" <gi...@apache.org>.
hudi-bot commented on PR #12202:
URL: https://github.com/apache/hudi/pull/12202#issuecomment-2455733425

   <!--
   Meta data
   {
     "version" : 1,
     "metaDataEntries" : [ {
       "hash" : "557fd2761e0817b6be38ecd5a5eb7a1a52f2bb46",
       "status" : "SUCCESS",
       "url" : "https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=1491",
       "triggerID" : "557fd2761e0817b6be38ecd5a5eb7a1a52f2bb46",
       "triggerType" : "PUSH"
     } ]
   }-->
   ## CI report:
   
   * 557fd2761e0817b6be38ecd5a5eb7a1a52f2bb46 Azure: [SUCCESS](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=1491) 
   
   <details>
   <summary>Bot commands</summary>
     @hudi-bot supports the following commands:
   
    - `@hudi-bot run azure` re-run the last Azure build
   </details>


-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org