milleruntime commented on a change in pull request #2171:
URL: https://github.com/apache/accumulo/pull/2171#discussion_r656541916
##########
File path: core/src/test/resources/accumulo2.properties
##########
@@ -24,6 +24,6 @@ instance.volumes=hdfs://localhost:8020/accumulo123
instance.zookeeper.host=myhost123:2181
table.durability=flush
tserver.memory.maps.native.enabled=false
-tserver.walog.max.size=256M
+tserver.wal.max.size=256M
Review comment:
Good catch.
##########
File path:
server/base/src/main/java/org/apache/accumulo/server/master/recovery/HadoopLogCloser.java
##########
@@ -36,7 +36,7 @@
public HadoopLogCloser() {
log.warn("{} has been deprecated. Please update property {} to {}
instead.",
- getClass().getName(),
Property.MANAGER_WALOG_CLOSER_IMPLEMETATION.getKey(),
+ getClass().getName(),
Property.MANAGER_WAL_CLOSER_IMPLEMENTATION.getKey(),
Review comment:
I don't think you need to worry about resolving here. What you have is
fine since this whole class was just deprecated in main. You just replaced the
new property with a newer one.
##########
File path: core/src/main/java/org/apache/accumulo/core/conf/Property.java
##########
@@ -360,25 +365,59 @@
+ " When there are more RFiles than this setting multiple passes
must be"
+ " made, which is slower. However opening too many RFiles at once
can cause"
+ " problems."),
+ TSERV_WAL_MAX_REFERENCED("tserver.wal.max.referenced", "3",
PropertyType.COUNT,
+ "When a tablet server has more than this many write ahead logs, any
tablet referencing older "
+ + "logs over this threshold is minor compacted. Also any tablet
referencing this many "
+ + "logs or more will be compacted."),
+ @Deprecated(since = "2.1.0")
+ @ReplacedBy(property = Property.TSERV_WAL_MAX_REFERENCED)
TSERV_WALOG_MAX_REFERENCED("tserver.walog.max.referenced", "3",
PropertyType.COUNT,
"When a tablet server has more than this many write ahead logs, any
tablet referencing older "
+ "logs over this threshold is minor compacted. Also any tablet
referencing this many "
+ "logs or more will be compacted."),
+ TSERV_WAL_MAX_SIZE("tserver.wal.max.size", "1G", PropertyType.BYTES,
+ "The maximum size for each write-ahead log. See comment for property"
+ + " tserver.memory.maps.max"),
+ @Deprecated(since = "2.1.0")
+ @ReplacedBy(property = Property.TSERV_WAL_MAX_SIZE)
TSERV_WALOG_MAX_SIZE("tserver.walog.max.size", "1G", PropertyType.BYTES,
"The maximum size for each write-ahead log. See comment for property"
+ " tserver.memory.maps.max"),
+ TSERV_WAL_MAX_AGE("tserver.wal.max.age", "24h", PropertyType.TIMEDURATION,
+ "The maximum age for each write-ahead log."),
+ @Deprecated(since = "2.1.0")
+ @ReplacedBy(property = Property.TSERV_WAL_MAX_AGE)
TSERV_WALOG_MAX_AGE("tserver.walog.max.age", "24h",
PropertyType.TIMEDURATION,
"The maximum age for each write-ahead log."),
+
TSERV_WAL_TOLERATED_CREATION_FAILURES("tserver.wal.tolerated.creation.failures",
"50",
+ PropertyType.COUNT,
+ "The maximum number of failures tolerated when creating a new
write-ahead"
+ + " log. Negative values will allow unlimited creation failures.
Exceeding this"
+ + " number of failures consecutively trying to create a new
write-ahead log"
+ + " causes the TabletServer to exit."),
+ @Deprecated(since = "2.1.0")
+ @ReplacedBy(property = Property.TSERV_WAL_TOLERATED_CREATION_FAILURES)
TSERV_WALOG_TOLERATED_CREATION_FAILURES("tserver.walog.tolerated.creation.failures",
"50",
PropertyType.COUNT,
"The maximum number of failures tolerated when creating a new
write-ahead"
+ " log. Negative values will allow unlimited creation failures.
Exceeding this"
+ " number of failures consecutively trying to create a new
write-ahead log"
+ " causes the TabletServer to exit."),
+ TSERV_WAL_TOLERATED_WAIT_INCREMENT("tserver.wal.tolerated.wait.increment",
"1000ms",
+ PropertyType.TIMEDURATION,
+ "The amount of time to wait between failures to create or write a
write-ahead log."),
+ @Deprecated(since = "2.1.0")
+ @ReplacedBy(property = Property.TSERV_WAL_TOLERATED_WAIT_INCREMENT)
TSERV_WALOG_TOLERATED_WAIT_INCREMENT("tserver.walog.tolerated.wait.increment",
"1000ms",
PropertyType.TIMEDURATION,
"The amount of time to wait between failures to create or write a
write-ahead log."),
// Never wait longer than 5 mins for a retry
+
TSERV_WAL_TOLERATED_MAXIMUM_WAIT_DURATION("tserver.wal.maximum.wait.duration",
"5m",
+ PropertyType.TIMEDURATION,
+ "The maximum amount of time to wait after a failure to create or write a
write-ahead log."),
+ // Never wait longer than 5 mins for a retry
+ @Deprecated(since = "2.1.0")
+ @ReplacedBy(property = Property.TSERV_WAL_TOLERATED_MAXIMUM_WAIT_DURATION)
Review comment:
Although the Property names are not sorted, we usually try to group
common ones together. You could put the new tserver ones all together with the
other "tserver.wal" properties. But I don't think this is a big deal since they
can be sorted in an IDE.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
