The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.
| Tag Class | Total number of occurrences | Tag strings used by tag class | 
|---|---|---|
| @todo | 0 | @todo | 
| TODO | 42 | TODO | 
Each tag is detailed below:
Number of occurrences found in the code: 42
| org.apache.giraph.TestGraphPartitioner | Line | 
|---|---|
| this is fragile (breaks with legit serialization changes) | 51 | 
| org.apache.giraph.block_app.framework.AbstractBlockFactory | Line | 
| - see if it should be deprecated | 191 | 
| org.apache.giraph.block_app.framework.internal.BlockWorkerContextLogic | Line | 
| add support through conf for postApplication, if needed. | 93 | 
| org.apache.giraph.block_app.framework.internal.BlockWorkerPieces | Line | 
| extract splitting logic into common utility | 132 | 
| org.apache.giraph.block_app.framework.piece.DefaultParentPiece | Line | 
| move to GiraphConstants | 71 | 
| Java8 - move these as default functions to VertexSender interface | 306 | 
| org.apache.giraph.block_app.migration.MigrationMasterCompute | Line | 
| --No comment-- | 191 | 
| org.apache.giraph.bsp.CentralizedServiceWorker | Line | 
| how to avoid this additional function | 217 | 
| org.apache.giraph.comm.messages.PointerListMessagesIterable | Line | 
| - if needed implement same for Safe as well | 60 | 
| org.apache.giraph.comm.messages.primitives.long_id.LongAbstractListStore | Line | 
| - can parallelize? populate with vertex ids already known | 78 | 
| - discussion | 152 | 
| org.apache.giraph.comm.messages.primitives.long_id.LongPointerListPerVertexStore | Line | 
| - any better way? | 104 | 
| org.apache.giraph.comm.messages.queue.AsyncMessageStoreWrapper | Line | 
| implement if LocalBlockRunner needs async message store | 162 | 
| org.apache.giraph.comm.netty.handler.SaslServerHandler | Line | 
| Move out into a separate, dedicated handler: ("FirstRequestHandler") or similar. | 60 | 
| Move out into a separate, dedicated handler. | 92 | 
| --No comment-- | 116 | 
| org.apache.giraph.examples.SimpleFailComputation | Line | 
| Change this behavior to WorkerContext | 37 | 
| org.apache.giraph.function.ObjectTransfer | Line | 
| Java8: cleanup callers | 92 | 
| Java8: cleanup callers | 118 | 
| org.apache.giraph.graph.GiraphTransferRegulator | Line | 
| implement defaults and configurable options for measuring the size of input <V> or <E> data per read vertex, and setting limits on totals per outgoing graph data Collection etc. (See GIRAPH-260) | 35 | 
| org.apache.giraph.master.BspServiceMaster | Line | 
| for very large number of partitions we might want to split this across multiple requests | 1174 | 
| org.apache.giraph.ooc.data.DiskBackedMessageStore | Line | 
| implement if LocalBlockRunner needs this message store | 146 | 
| org.apache.giraph.worker.BspServiceWorker | Line | 
| Make this check configurable by the user (i.e. search dmesg for problems). | 307 | 
| checkpointing messages along with vertices to avoid multiple loads of a partition when out-of-core is enabled. | 1375 | 
| checkpointing messages along with vertices to avoid multiple loads of a partition when out-of-core is enabled. | 1628 | 
| org.apache.giraph.worker.VertexInputSplitsCallable | Line | 
| this works for generic OutEdges, can create a better api to support more efficient translation for specific types | 216 | 
| org.apache.giraph.writable.kryo.HadoopKryo | Line | 
| Refactor this class into two separate classes depending on whether the reference tracking is enabled or disabled. | 78 | 
| org.apache.giraph.writable.kryo.KryoWritableWrapperTest | Line | 
| see if this can be supported, though this is a rare case: assertTrue(res == res.get(4)); | 91 | 
| org.apache.giraph.yarn.GiraphApplicationMaster | Line | 
| --No comment-- | 95 | 
| --No comment-- | 191 | 
| - what is the range for priority? how to decide? | 269 | 
| provide actual call back details | 340 | 
| org.apache.giraph.yarn.GiraphYarnClient | Line | 
| priority and queue Set the priority for the application master Priority pri = Records.newRecord(Priority.class); | 141 | 
| - what is the range for priority? how to decide? pri.setPriority(amPriority); appContext.setPriority(pri); | 144 | 
| --No comment-- | 158 | 
| --No comment-- | 245 | 
| --No comment-- | 258 | 
| REMOVE examples jar! | 438 | 
| Make constant | 439 | 
| add the runtime classpath needed for tests to work | 489 | 
| log4j? | 516 | 
| org.apache.giraph.yarn.YarnUtils | Line | 
| Separator | 187 |