[ 
https://issues.apache.org/jira/browse/TS-2122?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13734457#comment-13734457
 ] 

ASF subversion and git services commented on TS-2122:
-----------------------------------------------------

Commit 3c49b11ed3ab7c436db408af43be253f96c02ce4 in branch refs/heads/master 
from [~yunkai]
[ https://git-wip-us.apache.org/repos/asf?p=trafficserver.git;h=3c49b11 ]

TS-2123: Remove useless max_entries_per_buffer option

When the configurable value of max_entries_per_buffer in LogHost is
smaller that LogClient, it'll lead to log dropped silently.

Actually, after we fix https://issues.apache.org/jira/browse/TS-2122 issue,
the max_entries_per_buffer option is unnecessary now.

Let's remove it to avoid traps caused by it.

Signed-off-by: Yunkai Zhang <[email protected]>

                
> Enlarge the 64KB limitation of log buffer size
> ----------------------------------------------
>
>                 Key: TS-2122
>                 URL: https://issues.apache.org/jira/browse/TS-2122
>             Project: Traffic Server
>          Issue Type: Improvement
>          Components: Logging
>            Reporter: Yunkai Zhang
>         Attachments: 
> 0001-TS-2122-Enlarge-the-64KB-limitation-of-log-buffer-si.patch
>
>
> The Logging module use uint16_t(LogBuffer->m_state.s.m_offset/byte_count) to 
> store log buffer size:
> {code}
>   struct
>   {
>     uint16_t offset;              // buffer should be <= 64KB
>     uint16_t num_entries;         // number of entries in buffer
>     uint16_t byte_count;          // bytes in buffer
>     uint16_t full:1;              // not accepting more checkouts
>     uint16_t num_writers:15;      // number of writers
>   } s;
> {code}
> It's very dangerous for user, as there are not any warning when user breaks 
> the limitation.
> In our environment, we set the log_buffer_size to 512KB(and set 
> max_entries_per_buffer to 1000), it make ATS crashing.
> I'll give a patch to enlarge this limitation from 64KB to 4GB by merging 
> *offset* and *byte_count* variables into one, so that we can use uint32_t to 
> store the buffer size.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to