nsivabalan commented on code in PR #12214:
URL: https://github.com/apache/hudi/pull/12214#discussion_r1831414049
##########
hudi-common/src/main/java/org/apache/hudi/metadata/HoodieMetadataPayload.java:
##########
@@ -155,7 +156,7 @@ public class HoodieMetadataPayload implements
HoodieRecordPayload<HoodieMetadata
/**
* HoodieMetadata secondary index payload field ids
*/
- public static final String SECONDARY_INDEX_FIELD_RECORD_KEY = "recordKey";
+ public static final String SECONDARY_INDEX_RECORD_KEY_SEPARATOR = "$";
Review Comment:
here are my 2 cents.
lets add a suffix for the length of primary key.
for eg,
instead of "sfo$trip1", we can do "sfotrip1_5" where 5 represents the length
or primary key in this string.
so, this does not discriminate against any special characters in the primary
key nor secondary key.
wdyt
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
