You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-dev@hadoop.apache.org by "ZanderXu (Jira)" <ji...@apache.org> on 2022/10/08 11:03:00 UTC
[jira] [Created] (HDFS-16798) SerialNumberMap should decrease the current if the old already exist
ZanderXu created HDFS-16798:
-------------------------------
Summary: SerialNumberMap should decrease the current if the old already exist
Key: HDFS-16798
URL: https://issues.apache.org/jira/browse/HDFS-16798
Project: Hadoop HDFS
Issue Type: Bug
Reporter: ZanderXu
Assignee: ZanderXu
During looking into some code related XATTR, I found there is a bug in SerialNumberMap, as bellow:
{code:java}
public int get(T t) {
if (t == null) {
return 0;
}
Integer sn = t2i.get(t);
if (sn == null) {
sn = current.getAndIncrement();
if (sn > max) {
current.getAndDecrement();
throw new IllegalStateException(name + ": serial number map is full");
}
Integer old = t2i.putIfAbsent(t, sn);
if (old != null) {
// here: if the old is not null, we should decrease the current value.
return old;
}
i2t.put(sn, t);
}
return sn;
} {code}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-dev-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-help@hadoop.apache.org