一种分离队列条目的好方法

时间:2009-04-13 11:24:43

标签: java performance queue

我有一个线程池,从队列中提供任务。通常,少量线程能够将队列保持为空。偶尔,特别大的事件突发会使队列大小在一段时间内保持在零以上,但不会持续很长时间。

我担心的是重复的事件或带有废弃先前事件的数据。在高容量时,这样的事件可以在队列中共存很短的时间。我希望能够将这些内容混为一谈,这样我就可以减少浪费时间的工作。

将这样的队列混为一谈的好方法是什么?我可以在插入时通过从头到尾迭代并寻找替换的候选者来混淆,但这似乎太暴力了。如果您有代码或库建议,请记住我使用的是Java。

3 个答案:

答案 0 :(得分:5)

为什么不根据你的任务实现hashCode()和equals()。然后只需删除任务。例如。

queue.remove(task);
queue.offer(task);

然后你就不会有重复了。或者。或者。

if(!queue.contains(task)) {
   queue.offer(task);
}

如果任务已经在队列中,这将避免将任务排队。

答案 1 :(得分:2)

如果您使用LinkedHashMap,则可以保留条目添加到队列的顺序。

当匹配的条目到达时,我认为您想将其一些数据附加到原始队列条目。在这种情况下,您可以更新哈希对象,或使用HashMap.put(key, value)将排队项替换为新对象。 (我认为这保留了原始项目的顺序,但我没有测试过。)

请注意,您的代码需要显式同步对LinkedHashMap及其中的数据的读写访问权限。您不希望在另一个线程抓取它进行处理的同时更新队列中的项目。最简单的同步方法可能是访问LinkedHashMapCollections.synchronizedMap()

答案 2 :(得分:2)

这个混淆器似乎可以满足您的需求: https://github.com/GuillaumeArnaud/conflator

根据您的要求,可以更改实现以合并或将最新事件替换为现有事件(如果在混淆队列中存在)。

例如。对于以下内容,每个事件都实现为“Tick”,它定义了合并行为。

public class Tick implements Message<Tick> {

    private final String ticker;

    public long getInitialQuantity() {
        return initialQuantity;
    }

    private final long initialQuantity;

    public long getCurrentQuantity() {
        return currentQuantity;
    }

    private long currentQuantity;
    private int numberOfMerges;

    public String getTicker() {
        return ticker;
    }

    public Tick(String ticker, long quantity) {
        this.ticker = ticker;
        this.initialQuantity = quantity;
        this.currentQuantity = quantity;
    }

    @Override
    public String key() {
        return this.ticker;
    }

    @Override
    public String body() {
        return String.valueOf(currentQuantity);
    }

    @Override
    public boolean isMerged() {
        return this.initialQuantity != this.currentQuantity;
    }

    @Override
    public int mergesCount() {
        return numberOfMerges;
    }

    @Override
    public boolean isValid() {
        return false;
    }

    @Override
    public boolean merge(Tick message) {
        if (this.equals(message)) {
            this.currentQuantity += message.currentQuantity;
            numberOfMerges++;
            return true;
        }
        return false;
    }

    @Override
    public int hashCode() {
        return ticker.hashCode();
    }

    @Override
    public boolean equals(Object obj) {
        if (obj != null && obj instanceof Tick) {
            Tick other = (Tick) obj;
            return this.ticker.equals(other.getTicker());
        }
        return false;
    }

测试用例:

public class TickMergeTest {
    MultiValuedMapConflator conflator;

    @Test
    public void two_unmergeable_ticks_should_be_remain_unmergeable() {
        Tick tick1 = new Tick("GOOG", 100L);
        Tick tick2 = new Tick("AAPL", 120L);

        List<Tick> messages = conflator.merge(Lists.newArrayList(tick1, tick2));

        assertNotNull(messages);
        assertEquals(messages.size(), 2);
        assertEquals(Long.valueOf(messages.get(0).body()).longValue(), tick1.getCurrentQuantity());
        assertEquals(Long.valueOf(messages.get(1).body()).longValue(), tick2.getCurrentQuantity());
    }

    @Test(timeout = 1000)
    public void two_mergeable_ticks_should_be_merged() {
        Tick tick1 = new Tick("GOOG", 100L);
        Tick tick2 = new Tick("GOOG", 120L);

        List<Tick> messages = conflator.merge(Lists.newArrayList(tick1, tick2));

        assertNotNull(messages);
        assertEquals(messages.size(), 1);
        assertEquals(Long.valueOf(messages.get(0).body()).longValue(), tick1.getInitialQuantity() + tick2.getInitialQuantity());
    }

    @Test(timeout = 1000)
    public void should_merge_messages_on_same_key() throws InterruptedException {
        // given
        conflator.put(new Tick("GOOG", 100L));
        conflator.put(new Tick("GOOG", 120L));

        // test
        Thread.sleep(300); // waiting the conflation
        Message message = conflator.take();

        // check
        assertNotNull(message);
        assertEquals(Long.valueOf(message.body()).longValue(), 220L);
        assertTrue(message.isMerged());
    }

    @Test(timeout = 1000)
    public void should_not_merge_messages_on_diff_key() throws InterruptedException {
        // given
        conflator.put(new Tick("GOOG", 100L));
        conflator.put(new Tick("AAPL", 120L));

        // test
        Thread.sleep(300); // waiting the conflation
        Message message1 = conflator.take();
        Message message2 = conflator.take();

        // check
        assertNotNull(message1);
        assertNotNull(message2);

        assertEquals(Long.valueOf(message1.body()).longValue(), 100L);
        assertFalse(message1.isMerged());

        assertEquals(Long.valueOf(message2.body()).longValue(), 120L);
        assertFalse(message2.isMerged());

    }
    @Before
    public void setUp() {
        conflator = new MultiValuedMapConflator<Tick>(true);
    }

    @After
    public void tearDown() {
        conflator.stop();
    }
}