You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2022/01/14 09:48:07 UTC
[GitHub] [hudi] cdmikechen opened a new pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
cdmikechen opened a new pull request #4602:
URL: https://github.com/apache/hudi/pull/4602
## *Tips*
- *Thank you very much for contributing to Apache Hudi.*
- *Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.*
## What is the purpose of the pull request
JIRA <https://issues.apache.org/jira/browse/HUDI-3245>
Spark use CaseInsensitiveMap to get options, so that we can not get some configs in `HoodieStorageConfig` like `hoodie.parquet.writeLegacyFormat.enabled` and `hoodie.parquet.outputTimestampType`
We should convert uppercase letters to lowercase letters, so that it can not affect the related method.
## Brief change log
- replace `hoodie.parquet.writeLegacyFormat.enabled` to `hoodie.parquet.writelegacyformat.enabled`
- replace `hoodie.parquet.outputTimestampType` to `hoodie.parquet.outputtimestamptype`
## Verify this pull request
This pull request is already covered by existing tests
## Committer checklist
- [x] Has a corresponding JIRA in PR title & commit
- [x] Commit message is descriptive of the change
- [ ] CI is green
- [ ] Necessary doc changes done or have another open PR
- [ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
--
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
[GitHub] [hudi] hudi-bot commented on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot commented on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1012969853
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "PENDING",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 407ea19508c6906d08223c5d816392a60a10214d Azure: [PENDING](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241)
<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
[GitHub] [hudi] hudi-bot removed a comment on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot removed a comment on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1012967756
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "UNKNOWN",
"url" : "TBD",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 407ea19508c6906d08223c5d816392a60a10214d 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
[GitHub] [hudi] nsivabalan merged pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
nsivabalan merged pull request #4602:
URL: https://github.com/apache/hudi/pull/4602
--
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
[GitHub] [hudi] hudi-bot removed a comment on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot removed a comment on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1012981714
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "PENDING",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
}, {
"hash" : "a18347b62fc56cfcfc992533705425b4b94da731",
"status" : "UNKNOWN",
"url" : "TBD",
"triggerID" : "a18347b62fc56cfcfc992533705425b4b94da731",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 407ea19508c6906d08223c5d816392a60a10214d Azure: [PENDING](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241)
* a18347b62fc56cfcfc992533705425b4b94da731 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
[GitHub] [hudi] hudi-bot removed a comment on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot removed a comment on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1013016282
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "CANCELED",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
}, {
"hash" : "a18347b62fc56cfcfc992533705425b4b94da731",
"status" : "PENDING",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5242",
"triggerID" : "a18347b62fc56cfcfc992533705425b4b94da731",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 407ea19508c6906d08223c5d816392a60a10214d Azure: [CANCELED](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241)
* a18347b62fc56cfcfc992533705425b4b94da731 Azure: [PENDING](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5242)
<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
[GitHub] [hudi] hudi-bot removed a comment on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot removed a comment on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1012969853
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "PENDING",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 407ea19508c6906d08223c5d816392a60a10214d Azure: [PENDING](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241)
<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
[GitHub] [hudi] hudi-bot commented on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot commented on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1013016282
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "CANCELED",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
}, {
"hash" : "a18347b62fc56cfcfc992533705425b4b94da731",
"status" : "PENDING",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5242",
"triggerID" : "a18347b62fc56cfcfc992533705425b4b94da731",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 407ea19508c6906d08223c5d816392a60a10214d Azure: [CANCELED](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241)
* a18347b62fc56cfcfc992533705425b4b94da731 Azure: [PENDING](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5242)
<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
[GitHub] [hudi] hudi-bot removed a comment on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot removed a comment on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1012983989
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "CANCELED",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
}, {
"hash" : "a18347b62fc56cfcfc992533705425b4b94da731",
"status" : "UNKNOWN",
"url" : "TBD",
"triggerID" : "a18347b62fc56cfcfc992533705425b4b94da731",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 407ea19508c6906d08223c5d816392a60a10214d Azure: [CANCELED](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241)
* a18347b62fc56cfcfc992533705425b4b94da731 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
[GitHub] [hudi] hudi-bot commented on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot commented on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1012967756
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "UNKNOWN",
"url" : "TBD",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 407ea19508c6906d08223c5d816392a60a10214d 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
[GitHub] [hudi] hudi-bot commented on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot commented on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1013076011
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "DELETED",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
}, {
"hash" : "a18347b62fc56cfcfc992533705425b4b94da731",
"status" : "SUCCESS",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5242",
"triggerID" : "a18347b62fc56cfcfc992533705425b4b94da731",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* a18347b62fc56cfcfc992533705425b4b94da731 Azure: [SUCCESS](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5242)
<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
[GitHub] [hudi] cdmikechen commented on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
cdmikechen commented on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1014055807
@nsivabalan
Hi~ This PR may be a required patch for versions 0.10.1 and later. Can you contact the relevant committer/PMC for help, please?
In addition, there is a related [commit](https://github.com/apache/hudi/pull/4498) that also involves related issues, which may also need to be merged into 0.10.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 comment.
To unsubscribe, e-mail: commits-unsubscribe@hudi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [hudi] hudi-bot commented on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot commented on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1012983989
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "CANCELED",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
}, {
"hash" : "a18347b62fc56cfcfc992533705425b4b94da731",
"status" : "UNKNOWN",
"url" : "TBD",
"triggerID" : "a18347b62fc56cfcfc992533705425b4b94da731",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 407ea19508c6906d08223c5d816392a60a10214d Azure: [CANCELED](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241)
* a18347b62fc56cfcfc992533705425b4b94da731 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
[GitHub] [hudi] hudi-bot commented on pull request #4602: [HUDI-3245] Convert uppercase letters to lowercase in storage configs
Posted by GitBox <gi...@apache.org>.
hudi-bot commented on pull request #4602:
URL: https://github.com/apache/hudi/pull/4602#issuecomment-1012981714
<!--
Meta data
{
"version" : 1,
"metaDataEntries" : [ {
"hash" : "407ea19508c6906d08223c5d816392a60a10214d",
"status" : "PENDING",
"url" : "https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241",
"triggerID" : "407ea19508c6906d08223c5d816392a60a10214d",
"triggerType" : "PUSH"
}, {
"hash" : "a18347b62fc56cfcfc992533705425b4b94da731",
"status" : "UNKNOWN",
"url" : "TBD",
"triggerID" : "a18347b62fc56cfcfc992533705425b4b94da731",
"triggerType" : "PUSH"
} ]
}-->
## CI report:
* 407ea19508c6906d08223c5d816392a60a10214d Azure: [PENDING](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=5241)
* a18347b62fc56cfcfc992533705425b4b94da731 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