vinothchandar commented on code in PR #5716:
URL: https://github.com/apache/hudi/pull/5716#discussion_r929873650
##########
hudi-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadata.java:
##########
@@ -567,6 +564,16 @@ private synchronized void close(Pair<String, String>
partitionFileSlicePair) {
closeReader(readers);
}
+ /**
+ * Close and clear all the partitions readers.
+ */
+ private void closePartitionReaders() {
Review Comment:
this kind of refactoring also seems strictly not needed here in the same PR?
##########
hudi-common/src/main/java/org/apache/hudi/common/table/view/RocksDbBasedFileSystemView.java:
##########
@@ -199,6 +199,7 @@ protected void resetViewState() {
LOG.info("Deleting all rocksdb data associated with table filesystem
view");
rocksDB.close();
rocksDB = new RocksDBDAO(metaClient.getBasePath(),
config.getRocksdbBasePath());
+ schemaHelper.getAllColumnFamilies().forEach(rocksDB::addColumnFamily);
Review Comment:
please avoid unrelated changes in the same PR
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieWriteConfig.java:
##########
@@ -364,8 +364,8 @@ public class HoodieWriteConfig extends HoodieConfig {
public static final ConfigProperty<Boolean>
REFRESH_TIMELINE_SERVER_BASED_ON_LATEST_COMMIT = ConfigProperty
.key("hoodie.refresh.timeline.server.based.on.latest.commit")
- .defaultValue(false)
- .withDocumentation("Refresh timeline in timeline server based on latest
commit apart from timeline hash difference. By default (false), ");
+ .defaultValue(true)
Review Comment:
I don't know if we are ready for this flip yet. there are probably some open
JIRAs
--
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]
