You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi,I want to contribute to this issue.Can you just guide me through it?I do not have much experience with it but I am willing to learn and contribute:smile:.
You are welcome!)
For beginning, you can took at repository java-genproto, it contains java classes generated from proto specs. You can just update usage of TtlSettings in classes CreateTableSettings, AlterTableSettings and TableDescription
Hi,I read the proto specs and CreateTableSettings,AlterTableSetting.But I didn't understand how exactly are these classes using TtlSettings because it just returns ttlSetting object without performing any action/modifications.Then I looked up in BaseSession class where objects of CreateTableSettings, AlterTableSettings class set the expirySeconds and ColumnName for object of YdbTable.DateTypeColumnModeSettings.Builder class generated by proto.Can you help me understand how exactly do I need to update CreateTableSettings, AlterTableSettings ,what role do these classes play,and where does run_interval come into play.Thanks!!
Classes CreateTableSettings and AlterTableSettings are helpers for proto classes in YdbTable. Because direct usage of YdbTable is very unusual we made helpers for every proto message. Current implementation of TtlSettings and BaseSession supports only DataTypeColumnModeSettings and doesn't support ValueSinceUnixEpochModeSettings and run_interval_seconds. You can see it in this code which constructs CreateTableRequest message.
For resolving this issue we have to add new field to class TtlSettings and also add correct constructing of CreateTableRequest and AlterTableRequest in BaseSession's methods createTable and alterTable
https://github.com/ydb-platform/ydb/blob/main/ydb/public/api/protos/ydb_table.proto#L391
The text was updated successfully, but these errors were encountered: