This repository has been archived by the owner on Feb 8, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 59
Master #1
Open
ddppqq
wants to merge
1,490
commits into
Y2014
Choose a base branch
from
master
base: Y2014
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add documentation and examples for the .definefilter, .activatefilter, and .deactivatefilter directives in remap.config. These docs are specifically restricted to these particular names because (1) we should standardize on one variant for each of these directives to avoid end-user confusion and (2) I personally believe these variants to be the clearest and most understandable ("defflt", "useflt", "deuseflt", etc., are confusing or awkward). The .deletefilter directive is not documented, because I am not sure when or why it would be used.
Add line continuation to remap.config using a backslash at the end of a line as the continuation character. The handling for continuation was made in tokLine() by adding an optional third char parameter (`cont') which is the continuation character. This should make it usable in other contexts outside of remap.config also, for example other config file parsers. This implementation is not very intelligent, as it only checks for the backslash immediately preceding the newline and does not handle any whitespace. The backslash and newline are converted into spaces, and the next line is appended. Example: .definefilter foo \ @action=allow \ @src_ip=127.0.0.1 is read and parsed in UrlRewrite::BuildTable() as: .definefilter foo @action=allow @src_ip=127.0.0.1
libloader has been considered as stable since it's donation, but it wasn't put into the build by default. We change this now, and we add a convention for how to add plugins in the future.
The Logging module use uint16_t(LogBuffer->m_state.s.m_offset/byte_count) to store log buffer size. 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. This patch enlarge the 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. Signed-off-by: Yunkai Zhang <[email protected]>
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]>
1) Avoid silently dropping log entries in write_ascii_logbuffer3(). 2) Remove overspill logic, and try to write log to disk as much as possible. 3) Simplify the code by removing unecessary *overspill* variables. Signed-off-by: Yunkai Zhang <[email protected]>
Signed-off-by: Yunkai Zhang <[email protected]>
Added 'arch' (Architecture) section. Added cache documents. Fixed many warnings. Added Tiered Storage proposal.
AC_SEARCH_LIBS always updates $LIBS, but AC_CHECK_LIB doesn't search properly. So we have to introduce a wrapper around AC_SEARCH_LIBS that preserves $LIBS so that we can properly test for libraries without linking them into everything. Apply the new TS_SEARCH_LIBRARY macro to the geoip_acl plugin so that libGeoIP is not linked into every binary.
We were passing a format string to SSLDiagnostic(), but never actually using it when we logged the error. Improve the ssl_multicert.config error logging so that we log the file and line of where we detected an error.
Looks like m4_ifval isn't present in older versions of m4, however m4_default is probably a better choice anyway.
Now, in LogHost::write (LogBuffer *lb), Another new LogBuffer object will be allocated to copy the content of *lb* parameter. But *lb* will be deleted by the upper function, LogBufferManager::flush_buffers(), after LogHost::write() return. Obviously, the copy is unnecessary, and may cause memory fragmentation. Let's optimize it. Signed-off-by: Yunkai Zhang <[email protected]>
Calling "make install" sets ${localstatedir} owner and group to "ats" user. Needlessly so. This patch fixes it.
Older automake (< 1.9?) doesn't emit $(builddir), but it's always "." anyway, so let's use that instead.
1) Use nonblock eventfd, so that we can tolerate write() failed with errno EAGANIN – which is acceptable as the signal receiver will be notified eventually in this case. 2) After using nonblock eventfd, read() will not block in wait(). So I use epoll_wait() to implement block behavior, just like timedwait(). 3) nonblock eventfd can fix a potential problem: if receiver didn't read() data immediately, senders might block in write(). Signed-off-by: Yunkai Zhang <[email protected]>
Signed-off-by: Yunkai Zhang <[email protected]>
will result in empty responses
Signed-off-by: Yunkai Zhang <[email protected]>
…message and other handle Reliable(TCP) request.
Ubuntu Saucy (13.10) will be out of beta soon and comes with many goodies: Most prominently: clang 3.4 (and also clang-format). Right now we're still throwing a bunch of warnings when compiling with clang 3.4, so I'm adding support now that we can work on it asap.
2 cluster parameters: proxy.config.cluster.flow_ctrl.min_send_wait_time proxy.config.cluster.flow_ctrl.max_send_wait_time
in mgmt/RecordsConfig.cc, add two miessed cluster parameters: proxy.config.cluster.ping_retries proxy.config.cluster.connections
what is the merge want to do? |
Nick Kew seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
merge