Tracking issues of RFC-5313: Remove Metakey

[Xuanwo]

• RFC-5313: Remove Metakey #5313

Tasks

• feat: Add {stat,list}_has_* to carry the metadata that backend returns #5318
• refactor: Remove metakey concept #5319
• Implement stat_has_* and list_has_* correctly for services

[geetanshjuneja]

@Xuanwo I can take this task.
Do I have to set stat_has_* and list_has_* fields in capability struct passed to fn call to set_native_capability in every service?

[Xuanwo]

@Xuanwo I can take this task. Do I have to set stat_has_* and list_has_* fields in capability struct passed to fn call to set_native_capability in every service?

Yes! Please set the flags based on the metadata we specified in stat and list. I really appreciate seeing you handle it.

[geetanshjuneja]

Please set the flags based on the metadata we specified in stat and list

Are you talking about the metadata struct in Rpstat which is the return type of stat? If yes then how to decide whether the flag is true or false.

[Xuanwo]

Are you talking about the metadata struct in Rpstat which is the return type of stat? If yes then how to decide whether the flag is true or false.

For example, s3 set the following keys during list:

opendal/core/src/services/s3/lister.rs

Lines 126 to 136 in 37fe821

	let mut meta = Metadata::new(EntryMode::from_path(&path));
	if let Some(etag) = &object.etag {
	meta.set_etag(etag);
	meta.set_content_md5(etag.trim_matches('"'));
	}
	meta.set_content_length(object.size);
	// object.last_modified provides more precise time that contains
	// nanosecond, let's trim them.
	meta.set_last_modified(parse_datetime_from_rfc3339(object.last_modified.as_str())?);

We should set list_has_{etag,content_md5,content_length,last_modified} to true.

[geetanshjuneja]

In some services in stat fn metadata is obtained using parse_into_metadata fn

https://github.com/apache/opendal/blob/86d6c42051a338142225c06a49d7ea7d019b0e92/core/src/raw/http_util/header.rs#L156C1-L201C2

In this case I have to set all stat_has_* flags to true?

[Xuanwo]

In this case I have to set all stat_has_* flags to true?

Most (those one set by this function), but not all.