ibessonov commented on code in PR #7878:
URL: https://github.com/apache/ignite-3/pull/7878#discussion_r2993877142
##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/HashIndexLocker.java:
##########
@@ -43,24 +43,26 @@ public class HashIndexLocker implements IndexLocker {
* Constructs the object.
*
* @param indexId An identifier of the index this locker is created for.
+ * @param partitionId Partition ID.
+ * @param unique Uniqueness flag.
* @param lockManager A lock manager to acquire locks in.
* @param indexRowResolver A convertor which derives an index key from
given table row.
*/
- public HashIndexLocker(int indexId, boolean unique, LockManager
lockManager, ColumnsExtractor indexRowResolver) {
- this.indexId = indexId;
+ public HashIndexLocker(int indexId, int partitionId, boolean unique,
LockManager lockManager, ColumnsExtractor indexRowResolver) {
Review Comment:
Current constructor looks more natural to me
--
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]
